[Differential] [Updated] D1116: Fix issue with kdev-cppcheck selecting wrong project
kfunk (Kevin Funk)
noreply at phabricator.kde.org
Fri Mar 11 07:37:31 UTC 2016
kfunk added a comment.
Could you maybe refactor that code altogether? ProjectPath is quite pointless anyway. It's never read in kdev-cppcheck, only `ProjectPath + ErrorFile` is used.
My idea: Get rid off ProjectPath, just *always* store the absolute path in ErrorFile. IMO we don't even need `findProjectForUrl` at all. If you do run `cppcheck /path/fo/file.cpp` (what we do), then the resulting XML always contains the absolute path anyway.
REPOSITORY
rKDEVCPPCHECK KDevelop CppCheck
REVISION DETAIL
https://phabricator.kde.org/D1116
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: parolanilsson, kfunk
Cc: kdevelop-devel
More information about the KDevelop-devel
mailing list