[PATCH] Enhance PPCallbacks::Defined callback to provide	SourceRange
    Kim Gräsman 
    kim.grasman at gmail.com
       
    Fri Jul 19 14:19:25 PDT 2013
    
    
  
John, Argyrios,
On Fri, Jul 19, 2013 at 8:45 PM, Thompson, John
<John_Thompson at playstation.sony.com> wrote:
>
>>It would be great if you could also add some unit tests for the recent
>> changes in unittests/Lex/PPCallbacksTest.cpp but this can be done
>> post-commit.
>
> Oh, I didn’t know about this test.  Thanks for pointing it out. Will do.
I wrote PPCallbacksTest originally, and looking back at it, it's very
centered around the InclusionDirective callback. I wonder if it should
change names into PPCallbacksInclusionDirectiveTest or something?
The setup is pretty heavy for InclusionDirective, so it might prove
painful to add new, simpler PPCallbacks tests to this suite.
Just a heads-up,
- Kim
    
    
More information about the cfe-commits
mailing list