r199460 - Refine diagnostic: attribute 'nonnull' applies to both functions and methods.

Ted Kremenek kremenek at apple.com
Fri Jan 17 07:25:17 PST 2014


Thank you!

> On Jan 17, 2014, at 6:44 AM, Aaron Ballman <aaron at aaronballman.com> wrote:
> 
> Yeah, it was as I feared, we had no tests for this at all. I've added
> one in 199496. :-)
> 
> ~Aaron
> 
>> On Fri, Jan 17, 2014 at 8:23 AM, Aaron Ballman <aaron at aaronballman.com> wrote:
>> Good catch, thank you!  We should probably have a test for this
>> though. Do you mind adding one, or would you like me to take a stab at
>> it?
>> 
>> ~Aaron
>> 
>>> On Thu, Jan 16, 2014 at 11:58 PM, Ted Kremenek <kremenek at apple.com> wrote:
>>> Author: kremenek
>>> Date: Thu Jan 16 22:58:29 2014
>>> New Revision: 199460
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=199460&view=rev
>>> Log:
>>> Refine diagnostic: attribute 'nonnull' applies to both functions and methods.
>>> 
>>> Modified:
>>>    cfe/trunk/include/clang/Basic/Attr.td
>>> 
>>> Modified: cfe/trunk/include/clang/Basic/Attr.td
>>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/Attr.td?rev=199460&r1=199459&r2=199460&view=diff
>>> ==============================================================================
>>> --- cfe/trunk/include/clang/Basic/Attr.td (original)
>>> +++ cfe/trunk/include/clang/Basic/Attr.td Thu Jan 16 22:58:29 2014
>>> @@ -680,7 +680,7 @@ def NoMips16 : InheritableAttr, TargetSp
>>> def NonNull : InheritableAttr {
>>>   let Spellings = [GNU<"nonnull">, CXX11<"gnu", "nonnull">];
>>>   let Subjects = SubjectList<[ObjCMethod, FunctionLike, HasFunctionProto],
>>> -                             WarnDiag, "ExpectedFunction">;
>>> +                             WarnDiag, "ExpectedFunctionOrMethod">;
>>>   let Args = [VariadicUnsignedArgument<"Args">];
>>>   let AdditionalMembers =
>>> [{bool isNonNull(unsigned idx) const {
>>> 
>>> 
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits



More information about the cfe-commits mailing list