D5605: Allow multiple categories with the same name

David Edmundson noreply at phabricator.kde.org
Thu Apr 27 15:15:47 UTC 2017


davidedmundson added inline comments.

INLINE COMMENTS

> atticaprovider.cpp:129
>              qCDebug(KNEWSTUFFCORE) << "Adding category: " << category.name() << category.displayName();
> -            mCategoryMap[category.name()] = category;
> +            mCategoryMap.insertMulti(category.name(), category);
>  

mCategoryMap is pre-populated with null categories in the constructor.

This looks like this now won't ever update them.

REPOSITORY
  R304 KNewStuff

REVISION DETAIL
  https://phabricator.kde.org/D5605

To: mart, #plasma, leinir, whiting
Cc: davidedmundson, plasma-devel, #frameworks, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170427/6f2f94c9/attachment.html>


More information about the Plasma-devel mailing list