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