D20714: akonadi: fix some clazy warnings

Sergio Martins noreply at phabricator.kde.org
Sat Apr 27 17:07:24 BST 2019


smartins added inline comments.

INLINE COMMENTS

> dfaure wrote in itemcreatehandler.cpp:402
> @smartins Interesting. This means this is a clazy false positive. Did I miss a caveat somewhere explaining that the change should only be made if empty vs null makes no difference, unlike here?
> 
> I'll revert and add a silence-clazy comment.

README-empty-qstringliteral.md says:

"Note: Beware that `QString().isNull()` is `true` while both `QStringLiteral().isEmpty()` and `QStringLiteral("").isNull()` are `false`.
So be sure not to introduce subtle bugs when doing such replacements."

REPOSITORY
  R165 Akonadi

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

To: dfaure, dvratil, vkrause
Cc: smartins, kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190427/6c46e5f0/attachment.html>


More information about the kde-pim mailing list