Remove duplicate content in copied @param tags

src/core/lombok/core/handlers/HandlerUtil.java

@@ -910,9 +910,13 @@ public class HandlerUtil {
 		}
 	}
 	
-	public static String stripLinesWithTagFromJavadoc(String javadoc, JavadocTag tag) {
+	public static String stripLinesWithTagFromJavadoc(String javadoc, JavadocTag... tags) {
 		if (javadoc == null || javadoc.isEmpty()) return javadoc;
-		return tag.pattern.matcher(javadoc).replaceAll("").trim();
+		String result = javadoc;
+		for (JavadocTag tag : tags) {
+			result = tag.pattern.matcher(result).replaceAll("").trim();
+		}
+		return result;
 	}
 	
 	public static String stripSectionsFromJavadoc(String javadoc) {
@@ -968,7 +972,7 @@ public class HandlerUtil {
 	
 	public static String addJavadocLine(String in, String line) {
 		if (in == null) return line;
-		if (in.endsWith("\n")) return in + line + "\n";
+		if (in.endsWith("\n")) return in + line;
 		return in + "\n" + line;
 	}
 
@@ -990,11 +994,14 @@ public class HandlerUtil {
 		String fieldBaseJavadoc = stripSectionsFromJavadoc(fieldJavadoc);
 		
 		String paramJavadoc = getParamJavadoc(fieldBaseJavadoc, paramName);
-		if (paramJavadoc == null) {
-			paramJavadoc = stripLinesWithTagFromJavadoc(fieldBaseJavadoc, JavadocTag.PARAM);
-			paramJavadoc = stripLinesWithTagFromJavadoc(paramJavadoc, JavadocTag.RETURN);
+		if (paramJavadoc != null) {
+			return paramJavadoc;
 		}
 		
-		return paramJavadoc;
+		String javadocWithoutTags = stripLinesWithTagFromJavadoc(fieldBaseJavadoc, JavadocTag.PARAM, JavadocTag.RETURN);
+		if (javadocWithoutTags != null) {
+			return "@param " + paramName + " " + javadocWithoutTags;
+		}
+		return null;
 	}
 }

src/core/lombok/eclipse/handlers/HandleConstructor.java

@@ -598,12 +598,12 @@ public class HandleConstructor {
 			String paramJavadoc = getConstructorParameterJavadoc(paramName, fieldJavadoc);
 			
 			if (paramJavadoc == null) {
-				paramJavadoc = "";
+				paramJavadoc = "@param " + paramName;
 			} else {
 				fieldDescriptionAdded = true;
 			}
 			
-			constructorJavadoc = addJavadocLine(constructorJavadoc, "@param " + paramName + " " + paramJavadoc);
+			constructorJavadoc = addJavadocLine(constructorJavadoc, paramJavadoc);
 		}
 		if (fieldDescriptionAdded) {
 			setDocComment(typeNode, constructorNode, constructorJavadoc);

src/core/lombok/javac/handlers/HandleConstructor.java

@@ -490,12 +490,12 @@ public class HandleConstructor {
 			String paramJavadoc = getConstructorParameterJavadoc(paramName, fieldJavadoc);
 			
 			if (paramJavadoc == null) {
-				paramJavadoc = "";
+				paramJavadoc = "@param " + paramName;
 			} else {
 				fieldDescriptionAdded = true;
 			}
 			
-			constructorJavadoc = addJavadocLine(constructorJavadoc, "@param " + paramName + " " + paramJavadoc);
+			constructorJavadoc = addJavadocLine(constructorJavadoc, paramJavadoc);
 		}
 		if (fieldDescriptionAdded) {
 			setDocComment(cu, constructor, constructorJavadoc);

test/transform/resource/after-delombok/ConstructorJavadoc.java

@@ -16,7 +16,7 @@ public class ConstructorJavadoc {
 	/**
 	 * Creates a new {@code ConstructorJavadoc} instance.
 	 *
-	 * @param fieldName Some text
+	 * @param fieldName Hello, World1
 	 * @param fieldName2 Sky is blue
 	 */
 	@java.lang.SuppressWarnings("all")

test/transform/resource/after-ecj/ConstructorJavadoc.java

@@ -5,7 +5,7 @@ public @AllArgsConstructor class ConstructorJavadoc {
   /**
    * Creates a new {@code ConstructorJavadoc} instance.
    * 
-   * @param fieldName Some text
+   * @param fieldName Hello, World1
    * @param fieldName2 Sky is blue
    */
   public @java.lang.SuppressWarnings("all") @lombok.Generated ConstructorJavadoc(final int fieldName, final int fieldName2) {