Changeset View
Changeset View
Standalone View
Standalone View
src/tagcache.h
Show All 36 Lines | |||||
37 | * A tag cache | 37 | * A tag cache | ||
38 | */ | 38 | */ | ||
39 | class TagCache : public QObject | 39 | class TagCache : public QObject | ||
40 | { | 40 | { | ||
41 | Q_OBJECT | 41 | Q_OBJECT | ||
42 | public: | 42 | public: | ||
43 | TagCache(); | 43 | TagCache(); | ||
44 | Akonadi::Tag getTagByGid(const QByteArray &gid) const; | 44 | Akonadi::Tag getTagByGid(const QByteArray &gid) const; | ||
45 | Akonadi::Tag getTagByName(const QString &name) const; | ||||
dvratil: The argument should be called `name` | |||||
45 | 46 | | |||
46 | private Q_SLOTS: | 47 | private Q_SLOTS: | ||
47 | void onTagAdded(const Akonadi::Tag &); | 48 | void onTagAdded(const Akonadi::Tag &); | ||
48 | void onTagChanged(const Akonadi::Tag &); | 49 | void onTagChanged(const Akonadi::Tag &); | ||
49 | void onTagRemoved(const Akonadi::Tag &); | 50 | void onTagRemoved(const Akonadi::Tag &); | ||
50 | void onTagsFetched(KJob *); | 51 | void onTagsFetched(KJob *); | ||
51 | 52 | | |||
52 | private: | 53 | private: | ||
53 | void retrieveTags(); | 54 | void retrieveTags(); | ||
54 | 55 | | |||
55 | QHash<Akonadi::Tag::Id, Akonadi::Tag> mCache; | 56 | QHash<Akonadi::Tag::Id, Akonadi::Tag> mCache; | ||
56 | QHash<QByteArray, Akonadi::Tag::Id> mGidCache; | 57 | QHash<QByteArray, Akonadi::Tag::Id> mGidCache; | ||
58 | QHash<QString, Akonadi::Tag::Id> mNameCache; | ||||
I would suggest QHash<QString, Akonadi::Tag::Id>, becuase the name is actually a QString, so no need to convert it to/from a QString. knauss: I would suggest QHash<QString, Akonadi::Tag::Id>, becuase the name is actually a QString, so no… | |||||
57 | Akonadi::Monitor *mMonitor; | 59 | Akonadi::Monitor *mMonitor; | ||
58 | }; | 60 | }; | ||
59 | 61 | | |||
60 | } | 62 | } | ||
61 | 63 | | |||
62 | #endif | 64 | #endif |
The argument should be called name