[Differential] [Commented On] D450: Fix problem with include paths with clang language parser
amccann (Andrew McCann)
noreply at phabricator.kde.org
Sun Oct 25 20:43:39 UTC 2015
amccann added a comment.
switched to continue. (Funny, because thats what I originally had, but i've seen people not like use of 'continue' so went with the indentation..)
Decided to not add any isDir() / exists() checks. Agreed that clang should handle 'bad paths' OK.
Thinking is: the nature of the bug here isn't that a path was 'bad' but that an empty path was more of a syntax problem to clang's argument parser.
REPOSITORY
rKDEVELOP KDevelop
REVISION DETAIL
https://phabricator.kde.org/D450
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: amccann, kfunk, mwolff
Cc: andric, kdevelop-devel
More information about the KDevelop-devel
mailing list