Change Request
    Aaron Ballman via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Thu Mar 10 10:01:03 PST 2016
    
    
  
On Thu, Mar 10, 2016 at 11:38 AM, Wes Witt via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> I would like to submit the attached changes for your approval.
Thank you for the patch!
This:
+  let Subjects = SubjectList<[Function, Var, CXXRecord,
ObjCInterface], WarnDiag,
+                             "ExpectedVariableOrFunction">;
should be using ExpectedFunctionVariableOrClass. It's not completely
correct since you're adding Objective-C interfaces, but it's a bit
better than dropping the class part.
Also, patch submissions should include test cases exercising the
differences in the patch, so you should add some tests.
Welcome!
~Aaron
    
    
More information about the cfe-commits
mailing list