[PATCH] D54349: [clang-tidy] new check 'readability-redundant-preprocessor'
Miklos Vajna via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Nov 11 13:04:56 PST 2018
vmiklos marked 2 inline comments as done.
vmiklos added inline comments.
Comment at: clang-tidy/readability/RedundantPreprocessorCheck.cpp:56-59
+ StringRef SourceText =
+ PP.getSourceManager(), PP.getLangOpts());
+ std::string Condition = getCondition(SourceText);
> I'm a little confused. To me, it seems like you acquired the condition already -- doesn't `ConditionRange` actually cover the, well, condition range? This is how I imagined it:
> #ifdef CUTE_PANDA_CUBS
> Why is there a need for `getCondition`? Is there any? If there is (maybe the acquired text contains other things), can you document it? I haven't played with `PPCallbacks` much, so I'm fine with being in the wrong.
ConditionRange covers more than what you expect:
#if FOO == 4
to find out if the condition of the `#if` is the same as a previous one, I want to extract just `FOO == 4` from that, then deal with that part similar to `#ifdef` and `#ifndef`, which are easier as you have a single Token for the condition. But you're right, I should add a comment explaining this.
More information about the cfe-commits