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