diff options
Diffstat (limited to 'src/main/java/ftbsc')
| -rw-r--r-- | src/main/java/ftbsc/lll/processor/LilleroProcessor.java | 18 | ||||
| -rw-r--r-- | src/main/java/ftbsc/lll/processor/annotations/Injector.java | 7 |
2 files changed, 10 insertions, 15 deletions
diff --git a/src/main/java/ftbsc/lll/processor/LilleroProcessor.java b/src/main/java/ftbsc/lll/processor/LilleroProcessor.java index 321116e..28e7217 100644 --- a/src/main/java/ftbsc/lll/processor/LilleroProcessor.java +++ b/src/main/java/ftbsc/lll/processor/LilleroProcessor.java @@ -9,8 +9,6 @@ import ftbsc.lll.processor.exceptions.MappingNotFoundException; import ftbsc.lll.processor.exceptions.MappingsFileNotFoundException; import ftbsc.lll.tools.DescriptorBuilder; import ftbsc.lll.tools.SrgMapper; -import org.objectweb.asm.tree.ClassNode; -import org.objectweb.asm.tree.MethodNode; import javax.annotation.processing.*; import javax.lang.model.SourceVersion; @@ -79,7 +77,7 @@ public class LilleroProcessor extends AbstractProcessor { /** * This checks whether a given class contains the requirements to be parsed into a Lillero injector. * It must have at least one method annotated with {@link Target}, and one method annotated with {@link Injector} - * that must be public, static and take in a {@link ClassNode} and a {@link MethodNode}. + * that must be public, static and take in a ClassNode and MethodNode from ObjectWeb's ASM library. * @param elem the element to check. * @return whether it can be converted into a valid {@link IInjector}. */ @@ -122,6 +120,12 @@ public class LilleroProcessor extends AbstractProcessor { .get(); //will never be null so can ignore warning } + /** + * Builds a {@link MethodSpec} for a public method whose body simply returns a {@link String}. + * @param name the name of the method + * @param returnString the {@link String} to return + * @return the built {@link MethodSpec} + */ private static MethodSpec buildStringReturnMethod(String name, String returnString) { return MethodSpec.methodBuilder(name) .addModifiers(Modifier.PUBLIC) @@ -231,12 +235,6 @@ public class LilleroProcessor extends AbstractProcessor { String injectorSimpleClassName = cl.getSimpleName().toString() + "Injector"; String injectorClassName = packageName + "." + injectorSimpleClassName; - MethodSpec methodDesc = MethodSpec.methodBuilder("methodDesc") - .addModifiers(Modifier.PUBLIC) - .returns(String.class) - .addCode("return $S;", targetMethodDescriptor) - .build(); - MethodSpec inject = MethodSpec.methodBuilder("inject") .addModifiers(Modifier.PUBLIC) .returns(void.class) @@ -258,7 +256,7 @@ public class LilleroProcessor extends AbstractProcessor { .addMethod(buildStringReturnMethod("reason", ann.reason())) .addMethod(buildStringReturnMethod("targetClass", targetClassSrgName.replace('/', '.'))) .addMethod(buildStringReturnMethod("methodName", targetMethodSrgName)) - .addMethod(methodDesc) + .addMethod(buildStringReturnMethod("methodDesc", targetMethodDescriptor)) .addMethod(inject) .build(); diff --git a/src/main/java/ftbsc/lll/processor/annotations/Injector.java b/src/main/java/ftbsc/lll/processor/annotations/Injector.java index df523ac..1e2d8dd 100644 --- a/src/main/java/ftbsc/lll/processor/annotations/Injector.java +++ b/src/main/java/ftbsc/lll/processor/annotations/Injector.java @@ -4,13 +4,10 @@ import java.lang.annotation.ElementType; import java.lang.annotation.Retention; import java.lang.annotation.RetentionPolicy; -import org.objectweb.asm.tree.ClassNode; -import org.objectweb.asm.tree.MethodNode; - /** * Marks a method as the injector method for purposes of generation. - * The method itself should be {@code public static}, and take in a {@link ClassNode} - * and a {@link MethodNode} as parameters. It will be discarded otherwise. + * The method itself should be {@code public static}, and take in a ClassNode and MethodNode + * (from the ObjectWeb ASM library) as parameters. It will be discarded otherwise. * It will also be discarded unless the containing class is not annotated with {@link Patch} * and no other method within the class is annotated with {@link Target}. * @see Patch |