-
Notifications
You must be signed in to change notification settings - Fork 91
Support Java overloaded instance methods #339
Support Java overloaded instance methods #339
Conversation
Introduced for overloaded methods detection.
return Extractors.isPublic(m) && | ||
Extractors.isConcrete(m) && | ||
m.getName().equals(invocation.name()) && | ||
(m.getParameterCount() + 1 == invocation.arity()) || (m.isVarArgs() && (m.getParameterCount() <= invocation.arity())); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like déjà vu... maybe TypeMatching.argumentsNumberMatches
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need to try it :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it doesn't fit.
@yloiseau This makes me think that This PR focuses only on instance methods, but once refactored a bit we could improve |
Recorded #341 for overloaded static methods. I'm ok with this PR, if you agree then I'll go ahead and merge. |
👍 |
Support Java overloaded instance methods
This supports resolving and invalidating Java overloaded instance methods.