[cfe-commits] r165518 - in /cfe/trunk: include/clang/AST/DeclObjC.h lib/AST/ASTContext.cpp lib/AST/DeclObjC.cpp
kyrtzidis at apple.com
Tue Oct 9 13:11:26 PDT 2012
Done in r165538, thanks for reviewing!
On Oct 9, 2012, at 12:17 PM, Richard Smith <richard at metafoo.co.uk> wrote:
> On Tue, Oct 9, 2012 at 11:53 AM, Benjamin Kramer <benny.kra at gmail.com> wrote:
> On 09.10.2012, at 20:34, Jordan Rose <jordan_rose at apple.com> wrote:
> > On Oct 9, 2012, at 11:19 , Argyrios Kyrtzidis <akyrtzi at gmail.com> wrote:
> >> + SmallVector<const ObjCMethodDecl *, 8> OverDecls;
> >> + Method->getOverriddenMethods(OverDecls);
> >> + for (SmallVector<const ObjCMethodDecl *, 8>::iterator
> >> + M = OverDecls.begin(),
> >> + MEnd = OverDecls.end();
> >> + M != MEnd; ++M)
> >> + Overridden.push_back(*M);
> > :-( Yuck. Darn C++ being all type-safe.
> Overridden.append(OverDecls.begin(), OverDecls.end())
> would be slightly less yucky. Should optimize down to a memcpy in this case.
> Likewise for the C++ virtual function case.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits