On Monday, July 02, 2012 7:46 PM, Jordan Rose wrote:
> On Jun 30, 2012, at 2:12 PM, Andy Gibbs wrote:
>
>> Part 3: Added extra diagnostic for invalid range in directive 
>> <verify-part3.diff>
>
> 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?

Cheers
Andy

 

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to