From 40466c3e2d7d26a43b58f008986fd0b84c986b27 Mon Sep 17 00:00:00 2001 From: Roel Spilker Date: Tue, 3 Aug 2010 02:54:03 +0200 Subject: Hey, the sneaky throws remover works, at least when run separately. --- src/core/lombok/bytecode/SneakyThrowsRemover.java | 80 +++++++++++++++++++++++ 1 file changed, 80 insertions(+) create mode 100644 src/core/lombok/bytecode/SneakyThrowsRemover.java (limited to 'src/core/lombok/bytecode/SneakyThrowsRemover.java') diff --git a/src/core/lombok/bytecode/SneakyThrowsRemover.java b/src/core/lombok/bytecode/SneakyThrowsRemover.java new file mode 100644 index 00000000..b816cd54 --- /dev/null +++ b/src/core/lombok/bytecode/SneakyThrowsRemover.java @@ -0,0 +1,80 @@ +package lombok.bytecode; + +import lombok.core.DiagnosticsReceiver; +import lombok.core.PostCompilerTransformation; + +import org.mangosdk.spi.ProviderFor; +import org.objectweb.asm.ClassAdapter; +import org.objectweb.asm.ClassReader; +import org.objectweb.asm.ClassVisitor; +import org.objectweb.asm.ClassWriter; +import org.objectweb.asm.MethodAdapter; +import org.objectweb.asm.MethodVisitor; +import org.objectweb.asm.Opcodes; +import org.objectweb.asm.commons.JSRInlinerAdapter; + +@ProviderFor(PostCompilerTransformation.class) +public class SneakyThrowsRemover implements PostCompilerTransformation { + private static class FixedClassWriter extends ClassWriter { + FixedClassWriter(ClassReader classReader, int flags) { + super(classReader, flags); + } + + @Override protected String getCommonSuperClass(String type1, String type2) { + //By default, ASM will attempt to live-load the class types, which will fail if meddling with classes in an + //environment with custom classloaders, such as Equinox. It's just an optimization; returning Object is always legal. + try { + return super.getCommonSuperClass(type1, type2); + } catch (Exception e) { + return "java/lang/Object"; + } + } + } + + protected byte[] fixJSRInlining(byte[] byteCode) { + ClassReader reader = new ClassReader(byteCode); + ClassWriter writer = new FixedClassWriter(reader, 0); + + ClassVisitor visitor = new ClassAdapter(writer) { + @Override public MethodVisitor visitMethod(int access, String name, String desc, String signature, String[] exceptions) { + return new JSRInlinerAdapter(super.visitMethod(access, name, desc, signature, exceptions), access, name, desc, signature, exceptions); + } + }; + + reader.accept(visitor, 0); + return writer.toByteArray(); + } + + @Override public byte[] applyTransformations(byte[] original, String className, DiagnosticsReceiver diagnostics) { + byte[] fixedByteCode = fixJSRInlining(original); + + ClassReader reader = new ClassReader(fixedByteCode); + ClassWriter writer = new FixedClassWriter(reader, ClassWriter.COMPUTE_MAXS | ClassWriter.COMPUTE_FRAMES); + + reader.accept(new ClassAdapter(writer) { + @Override public MethodVisitor visitMethod(int access, String name, String desc, String signature, String[] exceptions) { + return new SneakyThrowsRemoverVisitor(super.visitMethod(access, name, desc, signature, exceptions)); + } + }, 0); + return writer.toByteArray(); + } + + private class SneakyThrowsRemoverVisitor extends MethodAdapter { + SneakyThrowsRemoverVisitor(MethodVisitor mv) { + super(mv); + } + + @Override public void visitMethodInsn(int opcode, String owner, String name, String desc) { + boolean hit = true; + if (hit && opcode != Opcodes.INVOKESTATIC) hit = false; + if (hit && !"sneakyThrow".equals(name)) hit = false; + if (hit && !"lombok/Lombok".equals(owner)) hit = false; + if (hit && !"(Ljava/lang/Throwable;)Ljava/lang/RuntimeException;".equals(desc)) hit = false; + if (hit) { + super.visitInsn(Opcodes.ATHROW); + } else { + super.visitMethodInsn(opcode, owner, name, desc); + } + } + } +} -- cgit