-
Notifications
You must be signed in to change notification settings - Fork 1.1k
A reflect newMethodOverride #15035
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
Comments
We might need to pass a list of method symbols to override instead of just one to be able to override methods from multiple parents, e.g.: trait One:
def foo: (Any, Int) // oneFooSym
def foo(x: Int) = 1 // overload (so we can't just rely on the method name)
trait Two:
def foo: (Int, Any) // twoFooSym
// Generated by newClass:
class Impl extends One with Two:
// Generated by newMethodOverride(implSym, List(oneFooSym, twoFooSym)):
override def foo: (Int, Int) = ??? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Symbol.newMethodOverride
?) which given a method symbol in a parent takes care of generating a new symbol with the correct info and flags to override it.Originally posted by @smarter in #15024 (review)
The text was updated successfully, but these errors were encountered: