diff options
Diffstat (limited to 'src/lombok/javac/handlers')
-rw-r--r-- | src/lombok/javac/handlers/HandleGetter.java | 7 | ||||
-rw-r--r-- | src/lombok/javac/handlers/HandlePrintAST.java | 3 | ||||
-rw-r--r-- | src/lombok/javac/handlers/HandleSetter.java | 7 |
3 files changed, 10 insertions, 7 deletions
diff --git a/src/lombok/javac/handlers/HandleGetter.java b/src/lombok/javac/handlers/HandleGetter.java index da1b7107..5cc2c108 100644 --- a/src/lombok/javac/handlers/HandleGetter.java +++ b/src/lombok/javac/handlers/HandleGetter.java @@ -24,10 +24,10 @@ import com.sun.tools.javac.util.Name; @ProviderFor(JavacAnnotationHandler.class) public class HandleGetter implements JavacAnnotationHandler<Getter> { - @Override public void handle(AnnotationValues<Getter> annotation, JCAnnotation ast, JavacAST.Node annotationNode) { + @Override public boolean handle(AnnotationValues<Getter> annotation, JCAnnotation ast, JavacAST.Node annotationNode) { if ( annotationNode.up().getKind() != Kind.FIELD ) { annotationNode.addError("@Getter is only supported on a field."); - return; + return false; } String methodName = toGetterName((JCVariableDecl) annotationNode.up().get()); @@ -35,7 +35,7 @@ public class HandleGetter implements JavacAnnotationHandler<Getter> { if ( methodExists(methodName, annotationNode.up()) ) { annotationNode.addWarning( String.format("Not generating %s(): A method with that name already exists", methodName)); - return; + return false; } Getter getter = annotation.getInstance(); @@ -46,6 +46,7 @@ public class HandleGetter implements JavacAnnotationHandler<Getter> { JCMethodDecl getterMethod = createGetter(access, annotationNode.up(), annotationNode.getTreeMaker()); javacClassTree.defs = javacClassTree.defs.append(getterMethod); + return true; } private JCMethodDecl createGetter(int access, JavacAST.Node field, TreeMaker treeMaker) { diff --git a/src/lombok/javac/handlers/HandlePrintAST.java b/src/lombok/javac/handlers/HandlePrintAST.java index 23f4c6d6..3a738b4f 100644 --- a/src/lombok/javac/handlers/HandlePrintAST.java +++ b/src/lombok/javac/handlers/HandlePrintAST.java @@ -12,7 +12,8 @@ import lombok.javac.JavacAST.Node; @ProviderFor(JavacAnnotationHandler.class) public class HandlePrintAST implements JavacAnnotationHandler<PrintAST> { - @Override public void handle(AnnotationValues<PrintAST> annotation, JCAnnotation ast, Node annotationNode) { + @Override public boolean handle(AnnotationValues<PrintAST> annotation, JCAnnotation ast, Node annotationNode) { annotationNode.up().traverse(new JavacASTVisitor.JavacASTPrinter()); + return true; } } diff --git a/src/lombok/javac/handlers/HandleSetter.java b/src/lombok/javac/handlers/HandleSetter.java index 0fbfe8ae..da4e9ff0 100644 --- a/src/lombok/javac/handlers/HandleSetter.java +++ b/src/lombok/javac/handlers/HandleSetter.java @@ -26,10 +26,10 @@ import com.sun.tools.javac.util.Name; @ProviderFor(JavacAnnotationHandler.class) public class HandleSetter implements JavacAnnotationHandler<Setter> { - @Override public void handle(AnnotationValues<Setter> annotation, JCAnnotation ast, Node annotationNode) { + @Override public boolean handle(AnnotationValues<Setter> annotation, JCAnnotation ast, Node annotationNode) { if ( annotationNode.up().getKind() != Kind.FIELD ) { annotationNode.addError("@Setter is only supported on a field."); - return; + return false; } JCVariableDecl fieldNode = (JCVariableDecl) annotationNode.up().get(); @@ -39,7 +39,7 @@ public class HandleSetter implements JavacAnnotationHandler<Setter> { annotationNode.addWarning( String.format("Not generating %s(%s %s): A method with that name already exists", methodName, fieldNode.vartype, fieldNode.name)); - return; + return false; } Setter setter = annotation.getInstance(); @@ -50,6 +50,7 @@ public class HandleSetter implements JavacAnnotationHandler<Setter> { JCMethodDecl setterMethod = createSetter(access, annotationNode.up(), annotationNode.getTreeMaker()); javacClassTree.defs = javacClassTree.defs.append(setterMethod); + return true; } private JCMethodDecl createSetter(int access, JavacAST.Node field, TreeMaker treeMaker) { |