Internal refactoring. If we have a toMap, a toList makes sense (which is already needed internally).

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/csv/trunk@1560003 13f79535-47bb-0310-9956-ffa450edef68

Author: garydgregory

src/main/java/org/apache/commons/csv/CSVRecord.java

@@ -21,6 +21,7 @@
 import java.util.Arrays;
 import java.util.HashMap;
 import java.util.Iterator;
+import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
 
@@ -166,7 +167,7 @@ public boolean isSet(final String name) {
      * @return an iterator over the values of this record.
      */
     public Iterator<String> iterator() {
-        return Arrays.asList(values).iterator();
+        return toList().iterator();
     }
 
     /**
@@ -191,6 +192,16 @@ public int size() {
         return values.length;
     }
 
+    /**
+     * Convert the values to a List.
+     * 
+     * TODO: Maybe make this public?
+     * @return a new List
+     */
+    private List<String> toList() {
+        return Arrays.asList(values);
+    }
+
     /**
      * Converts this record into a Map.
      *