[PATCH] Correctly check for the main file in the presence of line markers
eli.friedman at gmail.com
Wed Aug 21 17:29:59 PDT 2013
On Wed, Aug 21, 2013 at 4:36 PM, Eli Friedman <eli.friedman at gmail.com>wrote:
> On Wed, Aug 21, 2013 at 2:53 PM, Richard Smith <richard at metafoo.co.uk>wrote:
>> I think isPresumedFromMainFile should be the more common choice, so maybe
>> should have the more obvious name. Perhaps call the new mechanism
>> isFromMainFile and rename the existing mechanism to isWrittenInMainFile?
>> I'm still not overjoyed with those names.
>> FWIW, all calls of isFromMainFile in lib/StaticAnalyzer look like they
>> should be using the new mechanism, for consistency when handling
>> preprocessed source. [That only leaves the calls in lib/Rewrite and
>> VerifyDiagnosticConsumer (both of which should stay as-is), and the one
>> caller of isInPrimaryFile that you didn't change (which seems reasonable to
>> switch over, assuming the #include_next implementation can cope with
>> presumed locations).]
> #include_next can't use presumed locations: evaluating it requires keeping
> track of state (specifically, which search directory the file was found in)
> which isn't recorded in line marker directives.
> The other feedback makes sense; thanks.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits