-
Notifications
You must be signed in to change notification settings - Fork 130
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
[23] ECJ accepts both boolean values and a default case in a switch #2916
Comments
Thanks, @jarthana this is a good one. Keep'em coming! :) |
FWIW, when I remove any of the three case statements, Javac crashes. I have a raised a defect with Oracle for this. |
Fixed by #2919 |
This one: https://bugs.openjdk.org/browse/JDK-8339602 ? Wow that was fast. Turned out my tests tip-toed narrowly around that case, otherwise Should I add that case to our suite anyway? |
The JDK issue has been closed as "not an issue". Don't know what to make of it :) |
Not an issue could mean - it got fixed otherwise somehow and is not reproducible on their tip. It would be good to check their recent build. I don't think they would close a compiler crash otherwise. |
Consider this code:
ECJ accepts this but Javac rejects this with error:
X.java:8: error: switch has both boolean values and a default label
I haven't checked with the spec on this, though.
The text was updated successfully, but these errors were encountered: