Array types are also a problem.

immutables · Oct 6, 2017 · 46e7335 · 46e7335
1 parent a1fd025
commit 46e7335
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/...c/org/immutables/fixture/builder/attribute_builders/ThirdPartyImmutableWithPrimitive.java b/...c/org/immutables/fixture/builder/attribute_builders/ThirdPartyImmutableWithPrimitive.java
@@ -12,6 +12,12 @@ public int getValue() {
         return value;
     }
 
+    public int[] getArrayValue() {
+        int[] x = {value};
+        return x;
+    }
+
+
     public static Builder generateNewBuilder() {
         return new Builder();
     }

diff --git a/value-processor/src/org/immutables/value/processor/meta/AttributeBuilderReflection.java b/value-processor/src/org/immutables/value/processor/meta/AttributeBuilderReflection.java
@@ -424,7 +424,9 @@ protected static boolean isPossibleCopyMethod(ValueAttribute valueAttribute,
         } else if (onValueType
             && candidateCopyMethod.getParameters().size() == 0
             && !candidateCopyMethod.getModifiers().contains(Modifier.STATIC)) {
-          return !candidateCopyMethod.getReturnType().getKind().isPrimitive();
+
+          TypeKind kind = candidateCopyMethod.getReturnType().getKind();
+          return !kind.isPrimitive() && kind != TypeKind.ARRAY;
         }
       } else if (!onValueType && possibleCopyMethod.getKind() == ElementKind.CONSTRUCTOR) {