D15797: [kdev-clazy] : use canonical paths

[Milian Wolff]

mwolff requested changes to this revision.
mwolff added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> jobparameters.cpp:168
>  
> -        if (m_checkPath == projectRootPath) {
> +        if (m_checkPath == projectRootPath || canonicalPathToCheck == projectCanonicalRootPath) {
>              m_sources = allFiles;

you can remove the previous check, since the second necessarily has to include it

> jobparameters.cpp:175
>                          m_sources.clear();
> +                        // handle this separately so users don't see a filename they might not be expecting to see.
>                          m_sources += m_checkPath;

we don't do this anywhere else - I'd suggest you always operate on canonical paths

note that you may want to pretty print paths before showing them to the user anyways. there's a function somewhere (Project?) that takes a path and turns it into a relative-to-project-root string ready for consumption by the user

REPOSITORY
  R32 KDevelop

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

To: rjvbb, #kdevelop, antonanikin, mwolff
Cc: 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/20190115/3d05282f/attachment.html>