[trivial] improving consistency between javac vs. ecj output

wasn't worth an issue on the tracker: javac and ecj handlers for static constructors would differ; ecjs would specify the return type and constructor invocation using fully qualified types, whereas the handler for javac did not.

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

@@ -459,22 +459,15 @@ public class HandleConstructor {
 		
 		ListBuffer<JCTypeParameter> typeParams = new ListBuffer<JCTypeParameter>();
 		ListBuffer<JCVariableDecl> params = new ListBuffer<JCVariableDecl>();
-		ListBuffer<JCExpression> typeArgs1 = new ListBuffer<JCExpression>();
-		ListBuffer<JCExpression> typeArgs2 = new ListBuffer<JCExpression>();
 		ListBuffer<JCExpression> args = new ListBuffer<JCExpression>();
 		
 		if (!type.typarams.isEmpty()) {
 			for (JCTypeParameter param : type.typarams) {
-				typeArgs1.append(maker.Ident(param.name));
-				typeArgs2.append(maker.Ident(param.name));
 				typeParams.append(maker.TypeParameter(param.name, param.bounds));
 			}
-			returnType = maker.TypeApply(maker.Ident(type.name), typeArgs1.toList());
-			constructorType = maker.TypeApply(maker.Ident(type.name), typeArgs2.toList());
-		} else {
-			returnType = maker.Ident(type.name);
-			constructorType = maker.Ident(type.name);
 		}
+		returnType = namePlusTypeParamsToTypeReference(maker, typeNode, type.typarams);
+		constructorType = namePlusTypeParamsToTypeReference(maker, typeNode, type.typarams);
 		
 		for (JavacNode fieldNode : fields) {
 			JCVariableDecl field = (JCVariableDecl) fieldNode.get();

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

@@ -0,0 +1,11 @@
+class ConstructorInner {
+	static class Inner {
+		@java.lang.SuppressWarnings("all")
+		private Inner() {
+		}
+		@java.lang.SuppressWarnings("all")
+		public static ConstructorInner.Inner of() {
+			return new ConstructorInner.Inner();
+		}
+	}
+}

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

@@ -0,0 +1,13 @@
+class ConstructorInner {
+  static @lombok.AllArgsConstructor(staticName = "of") class Inner {
+    private @java.lang.SuppressWarnings("all") Inner() {
+      super();
+    }
+    public static @java.lang.SuppressWarnings("all") ConstructorInner.Inner of() {
+      return new ConstructorInner.Inner();
+    }
+  }
+  ConstructorInner() {
+    super();
+  }
+}

test/transform/resource/before/ConstructorInner.java

@@ -0,0 +1,5 @@
+class ConstructorInner {
+	@lombok.AllArgsConstructor(staticName = "of")
+	static class Inner {
+	}
+}