Repository navigation
Prevent macro expansion in preprocessor __has_include test - #1548
i-garrison wants to merge 4 commits into
Conversation
|
This pull request changes some projects for the first time in this development cycle. An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch. Git patchFurther information are available in Common Build Issues - Missing version increments. |
f39212b to
c835132
Compare
|
Would it have worked to use Lexer.setInsideIncludeDirective? Presumably the lack of macro expansion is already handled for normal includes. |
EDIT: added a couple of tests after looking again and this change needs to be improved as it does not allow header name via macro, will have another look now. |
2f72fa8 to
85fd7a3
Compare
85fd7a3 to
48e809e
Compare
Thanks for the hint, just made a change which actually uses |
__has_includepreprocessor expression__has_include()argument as header file, note header navigation from argument is not implementedfixes __has_include fails with <linux/fs.h> #1547