Refactor LeafNodeVector

- Remove unnecessary ctor.
- Create unmodifiable list in ctor to reflect that the data
  is now immutable.
- Remove defaultNode field.

1 files changed, 16 insertions, 20 deletions

diff --git a/config-lib/src/main/java/com/yahoo/config/LeafNodeVector.java b/config-lib/src/main/java/com/yahoo/config/LeafNodeVector.java
index e6dd8f3091f..a5dee11637f 100644
--- a/config-lib/src/main/java/com/yahoo/config/LeafNodeVector.java
+++ b/config-lib/src/main/java/com/yahoo/config/LeafNodeVector.java
@@ -16,47 +16,43 @@ import java.util.List;
  */
 public class LeafNodeVector<REAL, NODE extends LeafNode<REAL>> extends NodeVector<NODE> {
 
-    NODE defaultNode;
+    private final List<REAL> realValues;
 
-    /**
-     * Creates a new vector with the given default node.
-     */
-    // TODO: remove this ctor when the library uses reflection via builders, and resizing won't be necessary
-    public LeafNodeVector(NODE defaultNode) {
+    // TODO: take class instead of default node
+    public LeafNodeVector(List<REAL> values, NODE defaultNode) {
         assert (defaultNode != null) : "The default node cannot be null";
 
-        this.defaultNode = defaultNode;
-        if (createNew() == null) {
+        if (createNew(defaultNode) == null) {
             throw new NullPointerException("Unable to duplicate the default node.");
         }
-    }
 
-    // TODO: take class instead of default node when the library uses reflection via builders
-    public LeafNodeVector(List<REAL> values, NODE defaultNode) {
-        this(defaultNode);
         for (REAL value : values) {
-            NODE node = createNew();
+            NODE node = createNew(defaultNode);
             node.value = value;
             vector.add(node);
         }
+        realValues = realList(vector);
     }
 
     /**
      * Creates a new Node by cloning the default node.
      */
     @SuppressWarnings("unchecked")
-    private NODE createNew() {
+    private NODE createNew(NODE defaultNode) {
         return (NODE) (defaultNode).clone();
     }
 
-    // TODO: create unmodifiable list in ctor when the library uses reflection via builders
-    @SuppressWarnings("unchecked")
-    public List<REAL> asList() {
-        List<REAL> ret = new ArrayList<REAL>();
+    private List<REAL> realList(List<NODE> nodes) {
+        List<REAL> reals = new ArrayList<REAL>();
         for(NODE node : vector) {
-            ret.add(node.value());
+            reals.add(node.value());
         }
-        return Collections.unmodifiableList(ret);
+        return Collections.unmodifiableList(reals);
+    }
+
+    @SuppressWarnings("unchecked")
+    public List<REAL> asList() {
+        return realValues;
     }
 
     // TODO: Try to eliminate the need for this method when we have moved FileAcquirer to the config library