D15797: [kdev-clazy] : use canonical paths

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Tue Jan 29 14:32:07 GMT 2019


kossebau added inline comments.

INLINE COMMENTS

> jobparameters.cpp:165
> +    const bool checkPathIsFile = pathInfo.isFile();
> +    const auto canonicalPathToCheck = checkPathIsFile ? pathInfo.canonicalFilePath() : QStringLiteral("");
>  

Why the `QStringLiteral("")` and not just a plain `QString()`?

REPOSITORY
  R32 KDevelop

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

To: rjvbb, #kdevelop, antonanikin, mwolff
Cc: kossebau, mwolff, antonanikin, kdevelop-devel, glebaccon, hase, antismap, iodelay, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190129/afbfa25e/attachment.html>


More information about the KDevelop-devel mailing list