using immutable list in FetchedValue

dfa1 · dfa1 · commit 0504013b16b7 · 2020-12-18T21:17:45.000+01:00
diff --git a/src/main/java/graphql/execution/FetchedValue.java b/src/main/java/graphql/execution/FetchedValue.java
@@ -4,7 +4,6 @@
 import graphql.GraphQLError;
 import graphql.Internal;
 
-import java.util.ArrayList;
 import java.util.List;
 import java.util.function.Consumer;
 
@@ -13,12 +12,12 @@ public class FetchedValue {
     private final Object fetchedValue;
     private final Object rawFetchedValue;
     private final Object localContext;
-    private final ImmutableList<GraphQLError> errors;
+    private final List<GraphQLError> errors;
 
     private FetchedValue(Object fetchedValue, Object rawFetchedValue, List<GraphQLError> errors, Object localContext) {
         this.fetchedValue = fetchedValue;
         this.rawFetchedValue = rawFetchedValue;
-        this.errors = ImmutableList.copyOf(errors);
+        this.errors = errors;
         this.localContext = localContext;
     }
 
@@ -75,7 +74,7 @@ public static class Builder {
         private Object fetchedValue;
         private Object rawFetchedValue;
         private Object localContext;
-        private List<GraphQLError> errors = new ArrayList<>();
+        private List<GraphQLError> errors = ImmutableList.of();
 
         public Builder fetchedValue(Object fetchedValue) {
             this.fetchedValue = fetchedValue;
@@ -93,7 +92,7 @@ public Builder localContext(Object localContext) {
         }
 
         public Builder errors(List<GraphQLError> errors) {
-            this.errors = errors;
+            this.errors = ImmutableList.copyOf(errors);
             return this;
         }
 
