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
WELD-1991 Fix invocation of overloaded method on decorator #1070
Conversation
Triggering build using a merge of 952528b on branch master: |
Build 918 is now running using a merge of 952528b on branch master: |
Build 918 outcome was SUCCESS using a merge of 952528b on branch master: |
return matching.get(0); | ||
} | ||
// Choose the most specific method | ||
// This does meet all requirements of JLS but it should work in most cases |
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 do not fully understand the comment. Perhaps it is wrong? If this code meets all the JLS requirements then it should work in all (not most) cases, right?
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.
not
is missing. I'll fix the comment.
Build 918 outcome was SUCCESS using a merge of 952528b on branch master: |
Triggering build using a merge of e3d6be6 on branch master: |
Build 940 is now running using a merge of e3d6be6 on branch master: |
Merged, thanks! |
No description provided.