D15797: [kdev-clazy] : use canonical paths
René J.V. Bertin
noreply at phabricator.kde.org
Thu Jan 9 21:38:19 GMT 2020
rjvbb added a comment.
Difficult to wrap my head around this, so much later!
Resuming your musings, aren't you in fact asking
> shouldn't it be simply `const auto canonicalPathToCheck = pathInfo.canonicalFilePath();`
and my answer to that: "could very well be".
I know that I tested the initial version of the patch, but I'm now beginning to wonder if I really checked the 3 scenarios with the final version. I think I did use the clazy plugin once or twice since, but the truth of the matter is that I hardly find use for it.
I'll try to remember to verify again (as is and with the mod above) but I wouldn't hold my breath until I get around to it :(
REPOSITORY
R32 KDevelop
REVISION DETAIL
https://phabricator.kde.org/D15797
To: rjvbb, #kdevelop, antonanikin, mwolff
Cc: kossebau, mwolff, antonanikin, kdevelop-devel, hmitonneau, christiant, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20200109/88e85937/attachment.html>
More information about the KDevelop-devel
mailing list