r186421 - This patch removes unused parameter allProperties and converts remaining

Jordan Rose jordan_rose at apple.com
Tue Jul 16 09:18:56 PDT 2013


Careful of odd formatting in a couple places. I don't think we have a standard for ArrayRef<Decl *> vs ArrayRef<Decl*>, but there should definitely be a space after the close-bracket.


On Jul 16, 2013, at 8:33 , Fariborz Jahanian <fjahanian at apple.com> wrote:

>   Decl *ActOnAtEnd(Scope *S, SourceRange AtEnd,
> -                   Decl **allMethods = 0, unsigned allNum = 0,
> -                   Decl **allProperties = 0, unsigned pNum = 0,
> -                   DeclGroupPtrTy *allTUVars = 0, unsigned tuvNum = 0);
> +                   ArrayRef<Decl *>allMethods = None,
> +                   ArrayRef<DeclGroupPtrTy>allTUVars = None);
>  


> -// Note: For class/category implemenations, allMethods/allProperties is
> -// always null.
> -Decl *Sema::ActOnAtEnd(Scope *S, SourceRange AtEnd,
> -                       Decl **allMethods, unsigned allNum,
> -                       Decl **allProperties, unsigned pNum,
> -                       DeclGroupPtrTy *allTUVars, unsigned tuvNum) {
> -
> +// Note: For class/category implementations, allMethods is always null.
> +Decl *Sema::ActOnAtEnd(Scope *S, SourceRange AtEnd, ArrayRef<Decl *>allMethods,
> +                       ArrayRef<DeclGroupPtrTy>allTUVars) {
>   if (getObjCContainerKind() == Sema::OCK_None)
>     return 0;




More information about the cfe-commits mailing list