Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

[Java17] Deal with Java 12 Switch statements in J2X #1268

Closed
Tracked by #1982
cdietrich opened this issue Dec 17, 2021 · 7 comments
Closed
Tracked by #1982

[Java17] Deal with Java 12 Switch statements in J2X #1268

cdietrich opened this issue Dec 17, 2021 · 7 comments
Milestone

Comments

@cdietrich
Copy link
Member

J2X exlodes with jdt java 12 switch statements
JavaAstFlattener
e.g. org.eclipse.xtend.core.tests.java8.javaconverter.Java8ConverterTest.testJava7()

package int thousand=1_000
package int million=1_000_000
package double d1=1000_000.0d
// long a1 = 0b1000_1010_0010_1101_1010_0001_0100_0101L;
package Map<String, List<String>> trades=new TreeMap()
def void processTrade(String t) {
	
	switch (t) {
		case MISSING,
		case MISSING,
		case MISSING,
		default :{
			
		}
	}
	try (var FileInputStream in=new FileInputStream("java7.txt")){
		System.out.println(in.read()) 
		throw new IllegalAccessException()
	} catch (IOException | IllegalAccessException e) {
		e.printStackTrace() 
	}
	
}
@cdietrich
Copy link
Member Author

check also other depre. in JavaASTFlattener

@cdietrich
Copy link
Member Author

maybe we also need a general mechanism to deal with stuff we dont support gracefully

@cdietrich cdietrich added this to the Release_2.27 milestone Jan 10, 2022
@cdietrich cdietrich modified the milestones: Release_2.27, Release_2.28 Apr 27, 2022
@cdietrich
Copy link
Member Author

Java 17 Supported moved to 2.28

@cdietrich cdietrich modified the milestones: Release_2.28, Release_2.29 Aug 22, 2022
@cdietrich
Copy link
Member Author

move to 2.30, likely wont fix

@cdietrich cdietrich modified the milestones: Release_2.29, Release_2.30 Nov 11, 2022
@szarnekow
Copy link
Contributor

I'd rather delete the entire Java2Xtend converter.

@miklossy
Copy link
Contributor

I also agree with @szarnekow

@cdietrich
Copy link
Member Author

wont fix

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants