[cfe-commits] r170365 - /cfe/trunk/include/clang/AST/DeclContextInternals.h

David Blaikie dblaikie at gmail.com
Mon Dec 17 12:51:54 PST 2012


On Mon, Dec 17, 2012 at 12:10 PM, Argyrios Kyrtzidis <akyrtzi at gmail.com> wrote:
> Author: akirtzidis
> Date: Mon Dec 17 14:10:50 2012
> New Revision: 170365
>
> URL: http://llvm.org/viewvc/llvm-project?rev=170365&view=rev
> Log:
> Just use begin()/end() instead of "&Vector[0], &Vector[0]+Vector.size()".
>
> No functionality change.
>
> Modified:
>     cfe/trunk/include/clang/AST/DeclContextInternals.h
>
> Modified: cfe/trunk/include/clang/AST/DeclContextInternals.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/DeclContextInternals.h?rev=170365&r1=170364&r2=170365&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/AST/DeclContextInternals.h (original)
> +++ cfe/trunk/include/clang/AST/DeclContextInternals.h Mon Dec 17 14:10:50 2012
> @@ -117,7 +117,7 @@
>      DeclsTy &Vector = *getAsVector();
>
>      // Otherwise, we have a range result.
> -    return DeclContext::lookup_result(&Vector[0], &Vector[0]+Vector.size());
> +    return DeclContext::lookup_result(Vector.begin(), Vector.end());

Perhaps change/update this API to take an ArrayRef instead?

>    }
>
>    /// HandleRedeclaration - If this is a redeclaration of an existing decl,
>
>
> _______________________________________________
> 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