Merge pull request #1235 from bulgakovalexander/bugfix/builderWithTolerate

Bugfix/builder with tolerate

1 files changed, 10 insertions, 6 deletions

diff --git a/src/core/lombok/eclipse/handlers/EclipseHandlerUtil.java b/src/core/lombok/eclipse/handlers/EclipseHandlerUtil.java
index 0ff5a7f6..59d8f587 100644
--- a/src/core/lombok/eclipse/handlers/EclipseHandlerUtil.java
+++ b/src/core/lombok/eclipse/handlers/EclipseHandlerUtil.java
@@ -1168,9 +1168,8 @@ public class EclipseHandlerUtil {
 							if (params < minArgs || params > maxArgs) continue;
 						}
 						
-						if (def.annotations != null) for (Annotation anno : def.annotations) {
-							if (typeMatches(Tolerate.class, node, anno.type)) continue top;
-						}
+						
+						if (isTolerate(node, def)) continue top;
 						
 						return getGeneratedBy(def) == null ? MemberExistsResult.EXISTS_BY_USER : MemberExistsResult.EXISTS_BY_LOMBOK;
 					}
@@ -1181,6 +1180,13 @@ public class EclipseHandlerUtil {
 		return MemberExistsResult.NOT_EXISTS;
 	}
 	
+	public static boolean isTolerate(EclipseNode node, AbstractMethodDeclaration def) {
+		if (def.annotations != null) for (Annotation anno : def.annotations) {
+			if (typeMatches(Tolerate.class, node, anno.type)) return true;
+		}
+		return false;
+	}
+	
 	/**
 	 * Checks if there is a (non-default) constructor. In case of multiple constructors (overloading), only
 	 * the first constructor decides if EXISTS_BY_USER or EXISTS_BY_LOMBOK is returned.
@@ -1198,9 +1204,7 @@ public class EclipseHandlerUtil {
 				if (def instanceof ConstructorDeclaration) {
 					if ((def.bits & ASTNode.IsDefaultConstructor) != 0) continue;
 					
-					if (def.annotations != null) for (Annotation anno : def.annotations) {
-						if (typeMatches(Tolerate.class, node, anno.type)) continue top;
-					}
+					if (isTolerate(node, def)) continue;
 					
 					return getGeneratedBy(def) == null ? MemberExistsResult.EXISTS_BY_USER : MemberExistsResult.EXISTS_BY_LOMBOK;
 				}