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
review: fix: add handling for UnresolvedReferenceBinding in ReferenceBuilder #5294
Conversation
Given the current design, the new method has to return null. Any empty |
src/test/resources/jdk/Class.java
Outdated
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'd be for minimizing this and recreating the bug :^)
I haven't found any small reproducer, so lets close this and live with the bug. |
I'd be in favor of deleting the test and merging it without any. The change is localized to an if that is not triggered by anything currently (or crashes), so we can't really make stuff worse. |
I am scared to add JDK code to the test resources. Any better suggestions? fixes #5290