[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