-
Notifications
You must be signed in to change notification settings - Fork 139
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
ECJ emits different inner class references for the same code #1635
ECJ emits different inner class references for the same code #1635
Conversation
261cd1a
to
43fd2d2
Compare
type references are tracked properly
5793378
to
895d440
Compare
@srikanth-sankaran : please next time avoid merging PR's that show errors. I know there are sometimes "checks" that are red without a good reason, but in general we are trying to keep the status green and so if anything is red it is most likely a sign of a problem with the PR. In doubt please just ask. |
@iloveeclipse I restarted the tests a couple of times only for the symptoms of the failing checks to shift around. I consulted with @jarthana and he asked me to go ahead since it "looks like the previous PRs that went in had the same problem." (Private chat) FWIW |
Ah, I see that what I took to be a temporary process glitch caused some grief around bundle numbers. Sorry about that! |
Oops! Sorry for the wrong and admittedly lazy advice. Off topic, it will be helpful if the failure reason (the version update) reported prominently rather then being buried in the log. |
…-jdt#1635) * Extract signature computation into a method of its own; Ensure nested type references are tracked properly * Fixes eclipse-jdt#1631
…-jdt#1635) * Extract signature computation into a method of its own; Ensure nested type references are tracked properly * Fixes eclipse-jdt#1631
What it does
Converge divergent signature management routines ensuring that we track nested member type references uniformly.
Fixes #1631
Author checklist