jyknight added a comment.

>> Your error looks correct to me -- "self" in a classmethod is not an 
>> instance, but the class itself. And while instances of X implement 
>> "Delegate", the Class does not.
> 
> Got it, thanks! We might need to add a flag to allow the old behavior 
> temporarily to accommodate our codebase while it's being updated.
Adding an explicit cast would be the simplest way to allow your code to 
continue being broken in the same way it was previously broken.  E.g.

  // TODO: fix the types here -- self is _not_ actually an id<Delegate>!
  y.d = static_cast<id<Delegate>>(self);


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67983/new/

https://reviews.llvm.org/D67983



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to