On Feb 11, 2014, at 9:36 , Aaron Ballman <[email protected]> wrote:

>> +    if (D->getFunctionType()) {
>> +      handleNonNullAttr(S, D, Attr);
>> +    } else {
>> +      S.Diag(Attr.getLoc(), diag::warn_attribute_nonnull_parm_no_args)
>> +        << D->getSourceRange();
>> +    }
> 
> Minor nit about the style -- shouldn't be using the curly braces here.

Uh...hm. I tried it without the curly braces, but it looked very strange with 
the << dangling there. Should I add a dummy comment to justify it?

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

Reply via email to