Skip to content

Commit dad5fec

Browse files
committed
Revert "Upgrade to Eclipse 2021-09"
This reverts commit f57832d. See gh-277
1 parent f57832d commit dad5fec

File tree

2 files changed

+10
-20
lines changed
  • spring-javaformat/spring-javaformat-formatter-eclipse-rewriter/src/main/java/io/spring/javaformat/formatter/eclipse/rewrite

2 files changed

+10
-20
lines changed

pom.xml

+2-2
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,12 @@
3131
<main.basedir>${basedir}</main.basedir>
3232
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
3333
<java.version>1.8</java.version>
34-
<eclipse.repository>https://download.eclipse.org/releases/2021-09/202109151000/</eclipse.repository>
34+
<eclipse.repository>https://download.eclipse.org/releases/2021-03/202103171000/</eclipse.repository>
3535
<eclipse.checkstyle.repository>https://checkstyle.org/eclipse-cs-update-site/</eclipse.checkstyle.repository>
3636
<tycho.disableP2Mirrors>true</tycho.disableP2Mirrors>
3737
<ant.version>1.8.1</ant.version>
3838
<ant-contrib.version>1.0b3</ant-contrib.version>
39-
<asm.version>7.3.1</asm.version>
39+
<asm.version>5.2</asm.version>
4040
<assertj.version>3.8.0</assertj.version>
4141
<checkstyle.version>8.45.1</checkstyle.version>
4242
<gradle.version>3.4</gradle.version>

spring-javaformat/spring-javaformat-formatter-eclipse-rewriter/src/main/java/io/spring/javaformat/formatter/eclipse/rewrite/EclipseRewriter.java

+8-18
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@
4242
* Internal build utility used to rewrite eclipse runtime classes.
4343
*
4444
* @author Phillip Webb
45-
* @author Andy Wilkinson
4645
*/
4746
public final class EclipseRewriter {
4847

@@ -76,7 +75,7 @@ public void rewrite(String file) throws IOException {
7675
private void rewrite(FileSystem zip) throws IOException {
7776
rewrite(zip, "org/eclipse/jdt/internal/formatter/DefaultCodeFormatter.class",
7877
DefaultCodeFormatterManipulator::new);
79-
rewrite(zip, "org/eclipse/osgi/util/NLS.class", NlsManipulator::new);
78+
rewrite(zip, "org/eclipse/osgi/util/NLS$1.class", NlsManipulator::new);
8079
}
8180

8281
private void rewrite(FileSystem zip, String name, Function<ClassWriter, ClassVisitor> manipulator)
@@ -101,7 +100,7 @@ public static void main(String[] args) throws Exception {
101100
private static class DefaultCodeFormatterManipulator extends ClassVisitor {
102101

103102
DefaultCodeFormatterManipulator(ClassVisitor visitor) {
104-
super(Opcodes.ASM7, visitor);
103+
super(Opcodes.ASM5, visitor);
105104
}
106105

107106
@Override
@@ -130,7 +129,7 @@ public MethodVisitor visitMethod(int access, String name, String desc, String si
130129
private static class DefaultCodeFormatterMethodManipulator extends MethodVisitor {
131130

132131
DefaultCodeFormatterMethodManipulator(MethodVisitor mv) {
133-
super(Opcodes.ASM7, mv);
132+
super(Opcodes.ASM5, mv);
134133
}
135134

136135
@Override
@@ -150,12 +149,12 @@ public void visitMethodInsn(int opcode, String owner, String name, String desc,
150149
private static class NlsManipulator extends ClassVisitor {
151150

152151
NlsManipulator(ClassVisitor visitor) {
153-
super(Opcodes.ASM7, visitor);
152+
super(Opcodes.ASM5, visitor);
154153
}
155154

156155
@Override
157156
public MethodVisitor visitMethod(int access, String name, String desc, String signature, String[] exceptions) {
158-
if ("<clinit>".equals(name)) {
157+
if ("run".equals(name) && desc.contains("Boolean")) {
159158
return new NslMethodManipulator(super.visitMethod(access, name, desc, signature, exceptions));
160159
}
161160
return super.visitMethod(access, name, desc, signature, exceptions);
@@ -172,26 +171,17 @@ private static class NslMethodManipulator extends MethodVisitor {
172171
private final MethodVisitor methodVisitor;
173172

174173
NslMethodManipulator(MethodVisitor mv) {
175-
super(Opcodes.ASM7, null);
174+
super(Opcodes.ASM5, null);
176175
this.methodVisitor = mv;
177176
}
178177

179178
@Override
180179
public void visitEnd() {
181180
MethodVisitor mv = this.methodVisitor;
182181
mv.visitCode();
183-
mv.visitInsn(Opcodes.ICONST_0);
184-
mv.visitTypeInsn(Opcodes.ANEWARRAY, "java/lang/Object");
185-
mv.visitFieldInsn(Opcodes.PUTSTATIC, "org/eclipse/osgi/util/NLS", "EMPTY_ARGS", "[Ljava/lang/Object;");
186182
mv.visitFieldInsn(Opcodes.GETSTATIC, "java/lang/Boolean", "TRUE", "Ljava/lang/Boolean;");
187-
mv.visitMethodInsn(Opcodes.INVOKEVIRTUAL, "java/lang/Boolean", "booleanValue", "()Z", false);
188-
mv.visitFieldInsn(Opcodes.PUTSTATIC, "org/eclipse/osgi/util/NLS", "ignoreWarnings", "Z");
189-
mv.visitTypeInsn(Opcodes.NEW, "java/lang/Object");
190-
mv.visitInsn(Opcodes.DUP);
191-
mv.visitMethodInsn(Opcodes.INVOKESPECIAL, "java/lang/Object", "<init>", "()V", false);
192-
mv.visitFieldInsn(Opcodes.PUTSTATIC, "org/eclipse/osgi/util/NLS", "ASSIGNED", "Ljava/lang/Object;");
193-
mv.visitInsn(Opcodes.RETURN);
194-
mv.visitMaxs(2, 0);
183+
mv.visitInsn(Opcodes.ARETURN);
184+
mv.visitMaxs(1, 1);
195185
mv.visitEnd();
196186
}
197187

0 commit comments

Comments
 (0)