author | zecke <zecke> | 2003-05-14 14:45:08 (UTC) |
---|---|---|
committer | zecke <zecke> | 2003-05-14 14:45:08 (UTC) |
commit | e4b4319e57b29cd00aa12ca21480df047d1b5e52 (patch) (unidiff) | |
tree | 3d8b9856aa513c026b19736f75d41e277acbf1ca | |
parent | e79fbade975dd5e0bf5d8fc649b755c11c7956c4 (diff) | |
download | opie-e4b4319e57b29cd00aa12ca21480df047d1b5e52.zip opie-e4b4319e57b29cd00aa12ca21480df047d1b5e52.tar.gz opie-e4b4319e57b29cd00aa12ca21480df047d1b5e52.tar.bz2 |
Guard against self assignemt as well
-rw-r--r-- | library/backend/palmtoprecord.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/library/backend/palmtoprecord.cpp b/library/backend/palmtoprecord.cpp index 3cfa874..8543559 100644 --- a/library/backend/palmtoprecord.cpp +++ b/library/backend/palmtoprecord.cpp | |||
@@ -1,141 +1,142 @@ | |||
1 | /********************************************************************** | 1 | /********************************************************************** |
2 | ** Copyright (C) 2000-2002 Trolltech AS. All rights reserved. | 2 | ** Copyright (C) 2000-2002 Trolltech AS. All rights reserved. |
3 | ** | 3 | ** |
4 | ** This file is part of the Qtopia Environment. | 4 | ** This file is part of the Qtopia Environment. |
5 | ** | 5 | ** |
6 | ** This file may be distributed and/or modified under the terms of the | 6 | ** This file may be distributed and/or modified under the terms of the |
7 | ** GNU General Public License version 2 as published by the Free Software | 7 | ** GNU General Public License version 2 as published by the Free Software |
8 | ** Foundation and appearing in the file LICENSE.GPL included in the | 8 | ** Foundation and appearing in the file LICENSE.GPL included in the |
9 | ** packaging of this file. | 9 | ** packaging of this file. |
10 | ** | 10 | ** |
11 | ** This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE | 11 | ** This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE |
12 | ** WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE. | 12 | ** WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE. |
13 | ** | 13 | ** |
14 | ** See http://www.trolltech.com/gpl/ for GPL licensing information. | 14 | ** See http://www.trolltech.com/gpl/ for GPL licensing information. |
15 | ** | 15 | ** |
16 | ** Contact info@trolltech.com if any conditions of this licensing are | 16 | ** Contact info@trolltech.com if any conditions of this licensing are |
17 | ** not clear to you. | 17 | ** not clear to you. |
18 | ** | 18 | ** |
19 | **********************************************************************/ | 19 | **********************************************************************/ |
20 | #include <qtopia/private/palmtoprecord.h> | 20 | #include <qtopia/private/palmtoprecord.h> |
21 | #include <qtopia/stringutil.h> | 21 | #include <qtopia/stringutil.h> |
22 | #include <qstringlist.h> | 22 | #include <qstringlist.h> |
23 | 23 | ||
24 | /*! \class Qtopia::PalmtopRecord palmtoprecord.h | 24 | /*! \class Qtopia::PalmtopRecord palmtoprecord.h |
25 | \brief The Qtopia::PalmtopRecord class is the base class for all PIM records. | 25 | \brief The Qtopia::PalmtopRecord class is the base class for all PIM records. |
26 | 26 | ||
27 | Provides unique id and category support for all PIM records. | 27 | Provides unique id and category support for all PIM records. |
28 | 28 | ||
29 | \ingroup qtopiaemb | 29 | \ingroup qtopiaemb |
30 | \ingroup qtopiadesktop | 30 | \ingroup qtopiadesktop |
31 | */ | 31 | */ |
32 | 32 | ||
33 | 33 | ||
34 | namespace Qtopia { | 34 | namespace Qtopia { |
35 | 35 | ||
36 | 36 | ||
37 | 37 | ||
38 | Record &Record::operator=( const Record &c ) | 38 | Record &Record::operator=( const Record &c ) |
39 | { | 39 | { |
40 | if(this == &c ) return *this; | ||
40 | mUid = c.mUid; | 41 | mUid = c.mUid; |
41 | mCats = c.mCats; | 42 | mCats = c.mCats; |
42 | customMap = c.customMap; | 43 | customMap = c.customMap; |
43 | return *this; | 44 | return *this; |
44 | } | 45 | } |
45 | 46 | ||
46 | void Record::setCategories( int single ) | 47 | void Record::setCategories( int single ) |
47 | { | 48 | { |
48 | if ( single == 0 ) | 49 | if ( single == 0 ) |
49 | return; | 50 | return; |
50 | mCats.resize(1); | 51 | mCats.resize(1); |
51 | mCats[0] = single; | 52 | mCats[0] = single; |
52 | } | 53 | } |
53 | 54 | ||
54 | // convenience methods provided for loading and saving to xml | 55 | // convenience methods provided for loading and saving to xml |
55 | QString Record::idsToString( const QArray<int> &catsUnsorted ) | 56 | QString Record::idsToString( const QArray<int> &catsUnsorted ) |
56 | { | 57 | { |
57 | QArray<int> cats = catsUnsorted; | 58 | QArray<int> cats = catsUnsorted; |
58 | cats.sort(); | 59 | cats.sort(); |
59 | 60 | ||
60 | QString str; | 61 | QString str; |
61 | for ( uint i = 0; i < cats.size(); i++ ) | 62 | for ( uint i = 0; i < cats.size(); i++ ) |
62 | if ( i == 0 ) | 63 | if ( i == 0 ) |
63 | str = QString::number( cats[int(i)] ); | 64 | str = QString::number( cats[int(i)] ); |
64 | else | 65 | else |
65 | str += ";" + QString::number( cats[int(i)] ); | 66 | str += ";" + QString::number( cats[int(i)] ); |
66 | 67 | ||
67 | return str; | 68 | return str; |
68 | } | 69 | } |
69 | 70 | ||
70 | // convenience methods provided for loading and saving to xml | 71 | // convenience methods provided for loading and saving to xml |
71 | QArray<int> Record::idsFromString( const QString &str ) | 72 | QArray<int> Record::idsFromString( const QString &str ) |
72 | { | 73 | { |
73 | QStringList catStrs = QStringList::split( ";", str ); | 74 | QStringList catStrs = QStringList::split( ";", str ); |
74 | QArray<int> cats( catStrs.count() ); | 75 | QArray<int> cats( catStrs.count() ); |
75 | uint i = 0; | 76 | uint i = 0; |
76 | for ( QStringList::ConstIterator it = catStrs.begin(); | 77 | for ( QStringList::ConstIterator it = catStrs.begin(); |
77 | it != catStrs.end(); ++it ) { | 78 | it != catStrs.end(); ++it ) { |
78 | cats[int(i)] = (*it).toInt(); | 79 | cats[int(i)] = (*it).toInt(); |
79 | i++; | 80 | i++; |
80 | } | 81 | } |
81 | return cats; | 82 | return cats; |
82 | } | 83 | } |
83 | 84 | ||
84 | /*! | 85 | /*! |
85 | Returns the string stored for the custom field \a key. | 86 | Returns the string stored for the custom field \a key. |
86 | Returns a null string if the field does not exist. | 87 | Returns a null string if the field does not exist. |
87 | */ | 88 | */ |
88 | QString Record::customField( const QString &key) const | 89 | QString Record::customField( const QString &key) const |
89 | { | 90 | { |
90 | if (customMap.contains(key)) | 91 | if (customMap.contains(key)) |
91 | return customMap[key]; | 92 | return customMap[key]; |
92 | 93 | ||
93 | return QString::null; | 94 | return QString::null; |
94 | } | 95 | } |
95 | 96 | ||
96 | /*! | 97 | /*! |
97 | Sets the string stored for the custom field \a key to \a value. | 98 | Sets the string stored for the custom field \a key to \a value. |
98 | */ | 99 | */ |
99 | void Record::setCustomField( const QString &key, const QString &value) | 100 | void Record::setCustomField( const QString &key, const QString &value) |
100 | { | 101 | { |
101 | // qWarning("setting custom " + key + " to " + value); | 102 | // qWarning("setting custom " + key + " to " + value); |
102 | if (customMap.contains(key)) | 103 | if (customMap.contains(key)) |
103 | customMap.replace(key, value); | 104 | customMap.replace(key, value); |
104 | else | 105 | else |
105 | customMap.insert(key, value); | 106 | customMap.insert(key, value); |
106 | 107 | ||
107 | // qWarning(QString("custom size %1").arg(customMap.count())); | 108 | // qWarning(QString("custom size %1").arg(customMap.count())); |
108 | } | 109 | } |
109 | 110 | ||
110 | /*! | 111 | /*! |
111 | Removes the custom field \a key. | 112 | Removes the custom field \a key. |
112 | */ | 113 | */ |
113 | void Record::removeCustomField(const QString &key) | 114 | void Record::removeCustomField(const QString &key) |
114 | { | 115 | { |
115 | customMap.remove(key); | 116 | customMap.remove(key); |
116 | } | 117 | } |
117 | 118 | ||
118 | QString Record::customToXml() const | 119 | QString Record::customToXml() const |
119 | { | 120 | { |
120 | //qWarning(QString("writing custom %1").arg(customMap.count())); | 121 | //qWarning(QString("writing custom %1").arg(customMap.count())); |
121 | QString buf(" "); | 122 | QString buf(" "); |
122 | for ( QMap<QString, QString>::ConstIterator cit = customMap.begin(); | 123 | for ( QMap<QString, QString>::ConstIterator cit = customMap.begin(); |
123 | cit != customMap.end(); ++cit) { | 124 | cit != customMap.end(); ++cit) { |
124 | // qWarning(".ITEM."); | 125 | // qWarning(".ITEM."); |
125 | buf += cit.key(); | 126 | buf += cit.key(); |
126 | buf += "=\""; | 127 | buf += "=\""; |
127 | buf += escapeString(cit.data()); | 128 | buf += escapeString(cit.data()); |
128 | buf += "\" "; | 129 | buf += "\" "; |
129 | } | 130 | } |
130 | return buf; | 131 | return buf; |
131 | } | 132 | } |
132 | 133 | ||
133 | void Record::dump( const QMap<int, QString> &map ) | 134 | void Record::dump( const QMap<int, QString> &map ) |
134 | { | 135 | { |
135 | QMap<int, QString>::ConstIterator it; | 136 | QMap<int, QString>::ConstIterator it; |
136 | for( it = map.begin(); it != map.end(); ++it ) | 137 | for( it = map.begin(); it != map.end(); ++it ) |
137 | qDebug("%d : %s", it.key(), it.data().local8Bit().data() ); | 138 | qDebug("%d : %s", it.key(), it.data().local8Bit().data() ); |
138 | } | 139 | } |
139 | 140 | ||
140 | } | 141 | } |
141 | 142 | ||