diff options
| author | Reinier Zwitserloot <reinier@tipit.to> | 2009-07-11 11:12:47 +0200 |
|---|---|---|
| committer | Reinier Zwitserloot <reinier@tipit.to> | 2009-07-11 11:12:47 +0200 |
| commit | f84f3a1bc8407160b82580300d2a58fd8c66aaa1 (patch) | |
| tree | a2e58110a042c8ce6d0c2fe022692976b90fb16d /src/lombok/javac/handlers | |
| parent | bc44e76b7d45da2301a1bb7d28d4e7e8fec4b94b (diff) | |
| download | lombok-f84f3a1bc8407160b82580300d2a58fd8c66aaa1.tar.gz lombok-f84f3a1bc8407160b82580300d2a58fd8c66aaa1.tar.bz2 lombok-f84f3a1bc8407160b82580300d2a58fd8c66aaa1.zip | |
The setter/getter handlers now mark themselves as not wanted to be called upon anymore when reporting errors.
They were logging 4 or more identical warnings per problem before this change.
Diffstat (limited to 'src/lombok/javac/handlers')
| -rw-r--r-- | src/lombok/javac/handlers/HandleGetter.java | 4 | ||||
| -rw-r--r-- | src/lombok/javac/handlers/HandleSetter.java | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/lombok/javac/handlers/HandleGetter.java b/src/lombok/javac/handlers/HandleGetter.java index 18e02a58..ed0370ac 100644 --- a/src/lombok/javac/handlers/HandleGetter.java +++ b/src/lombok/javac/handlers/HandleGetter.java @@ -91,7 +91,7 @@ public class HandleGetter implements JavacAnnotationHandler<Getter> { private boolean createGetterForField(AccessLevel level, Node fieldNode, Node errorNode, DiagnosticPosition pos, boolean whineIfExists) { if ( fieldNode.getKind() != Kind.FIELD ) { errorNode.addError("@Getter is only supported on a field."); - return false; + return true; } JCVariableDecl fieldDecl = (JCVariableDecl)fieldNode.get(); @@ -103,7 +103,7 @@ public class HandleGetter implements JavacAnnotationHandler<Getter> { case EXISTS_BY_USER: if ( whineIfExists ) errorNode.addWarning( String.format("Not generating %s(): A method with that name already exists", methodName)); - return false; + return true; default: case NOT_EXISTS: //continue with creating the getter diff --git a/src/lombok/javac/handlers/HandleSetter.java b/src/lombok/javac/handlers/HandleSetter.java index 817d3011..b1c37bb1 100644 --- a/src/lombok/javac/handlers/HandleSetter.java +++ b/src/lombok/javac/handlers/HandleSetter.java @@ -94,7 +94,7 @@ public class HandleSetter implements JavacAnnotationHandler<Setter> { private boolean createSetterForField(AccessLevel level, Node fieldNode, Node errorNode, DiagnosticPosition pos, boolean whineIfExists) { if ( fieldNode.getKind() != Kind.FIELD ) { fieldNode.addError("@Setter is only supported on a field."); - return false; + return true; } JCVariableDecl fieldDecl = (JCVariableDecl)fieldNode.get(); @@ -107,7 +107,7 @@ public class HandleSetter implements JavacAnnotationHandler<Setter> { if ( whineIfExists ) errorNode.addWarning( String.format("Not generating %s(%s %s): A method with that name already exists", methodName, fieldDecl.vartype, fieldDecl.name)); - return false; + return true; default: case NOT_EXISTS: //continue with creating the setter |