-
Notifications
You must be signed in to change notification settings - Fork 137
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
testBug434044_comment36 fails with Java 1.7 JLS since I20240606-1800 #2537
Comments
This difference:
is well in the range of acceptable fluctuation across compliance settings. @iloveeclipse do you still want me to make the test expectation version dependent, or will we stop testing at 1.7 "immediately"? |
If it is eas, fix, fix it, if not, leave it. I hope I can work on removal soon, but this weekend I'm offline. |
Still failing. What is the plan for this issue? Disable tests <=1.7 ? |
See #2536 |
But do capt
but it isn't correct behaviour? Its's capture of 2 - Do enough is just change it to 2 instead of 3? |
Those numbers are not guaranteed, they are a way to visualize different captures of the same wildcard as different. Other than that this number has no meaning. The point in this specific issue is: compilation at 1.7 vs 1.8 create different numbers of captures so the same capture gets different names. |
Fixed after Java 8 update. |
See https://download.eclipse.org/eclipse/downloads/drops4/I20240606-1800/testresults/html/org.eclipse.jdt.core.tests.compiler_ep433I-unit-cen64-gtk3-java17_linux.gtk.x86_64_17.html
Note: we probably should drop support for Java < Java 8 now, see #820.
I've also created #2536.
The text was updated successfully, but these errors were encountered: