D28008: Don't use deprecated QList::toSet

Daniel Vrátil noreply at phabricator.kde.org
Mon Mar 16 18:26:12 GMT 2020


dvratil requested changes to this revision.
dvratil added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> collectionfiltermodel.cpp:58
> +#else
> +        QSet<QString> contentMimeTypes(collection.contentMimeTypes().cbegin(), collection.contentMimeTypes().cend());
> +#endif

This will crash, since you are getting an iterator for a temporary (so begin and end iterators will belong to two different containers)

REPOSITORY
  R459 Akonadi Contacts

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

To: carewolf, mlaurent, vkrause, dvratil
Cc: dvratil, kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20200316/3b5d6374/attachment.html>


More information about the kde-pim mailing list