[cfe-commits] [Patch 3 of 7] -verify fixes and enhancement

Jordan Rose jordan_rose at apple.com
Tue Jul 3 17:37:38 PDT 2012


On Jul 3, 2012, at 3:23 PM, Andy Gibbs wrote:

> On Monday, July 02, 2012 8:45 PM, Jordan Rose wrote:
>> On Jul 2, 2012, at 11:40 , Andy Gibbs wrote:
>>> On Monday, July 02, 2012 7:46 PM, Jordan Rose wrote:
>>>> 
>>>> You should describe the new range syntax in the comment in
>>>> VerifyDiagnosticConsumer.h, but otherwise LGTM.
>>> 
>>> Silly question: did you notice that I did describe the new syntax?  Is it
>>> that you think I didn't explain it clearly enough?
>> 
>> Whoops, yes. I was expecting to see a line-of-code example for ranges just
>> like for "x-or-more" syntax, in the spirit of "a picture is worth a thousand
>> words". I just completely missed the single sentence that described the range
>> syntax.
>> 
>> (The "…or more" description was fine.)
> 
> Well, I had a few minutes and since I had to rebase the patches anyway, I've
> added a fuller description ;o)
> 
> Good to commit?

LGTM. Wondering about this case:

int x; // expected-note + {{...}}
...
// expected-note@<line> {{...}}

...but I'm willing to say that if you do this to yourself it shouldn't surprise you if it doesn't work.





More information about the cfe-commits mailing list