AbstractArrayConverter.parseElements(String) now returns a

List<String> instead of a raw List
apache · Dec 26, 2024 · a5ef8b1 · a5ef8b1
1 parent f498c9b
commit a5ef8b1
Show file tree

Hide file tree

Showing 11 changed files with 11 additions and 10 deletions.
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
@@ -35,6 +35,7 @@
       <action type="fix" dev="ggregory" due-to="Gary Gregory">Replace Maven CLIRR plugin with JApiCmp.</action>
       <action type="fix" dev="ggregory" due-to="Gary Gregory">Port to Java 1.4 Throwable APIs (!).</action>
       <action type="fix" dev="ggregory" due-to="Gary Gregory">Fix Javadoc generation on Java 8, 17, and 21.</action>
+      <action type="fix" dev="ggregory" due-to="Gary Gregory">AbstractArrayConverter.parseElements(String) now returns a List&lt;String&gt; instead of a raw List.</action>
       <!-- UPDATE -->
       <action dev="ggregory" type="update" due-to="Gary Gregory">Bump org.apache.commons:commons-parent from 47 to 78.</action>
       <action dev="ggregory" type="update" due-to="Gary Gregory">Bump Java requirement from Java 6 to 8.</action>

diff --git a/src/main/java/org/apache/commons/beanutils/converters/AbstractArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/AbstractArrayConverter.java
@@ -152,7 +152,7 @@ public AbstractArrayConverter(final Object defaultValue) {
      * @throws NullPointerException if <code>svalue</code>
      *  is <code>null</code>
      */
-    protected List parseElements(String svalue) {
+    protected List<String> parseElements(String svalue) {
 
         // Validate the passed argument
         if (svalue == null) {

diff --git a/src/main/java/org/apache/commons/beanutils/converters/BooleanArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/BooleanArrayConverter.java
@@ -216,7 +216,7 @@ public Object convert(final Class type, final Object value) {
         // sequence of values; see method AbstractArrayConverter.parseElements
         // for more information.
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final boolean[] results = new boolean[list.size()];
             for (int i = 0; i < results.length; i++) {
                 final String stringValue = (String) list.get(i);

diff --git a/src/main/java/org/apache/commons/beanutils/converters/ByteArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/ByteArrayConverter.java
@@ -122,7 +122,7 @@ public Object convert(final Class type, final Object value) {
         // Parse the input value as a String into elements
         // and convert to the appropriate type
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final byte[] results = new byte[list.size()];
             for (int i = 0; i < results.length; i++) {
                 results[i] = Byte.parseByte((String) list.get(i));

diff --git a/src/main/java/org/apache/commons/beanutils/converters/CharacterArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/CharacterArrayConverter.java
@@ -122,7 +122,7 @@ public Object convert(final Class type, final Object value) {
         // Parse the input value as a String into elements
         // and convert to the appropriate type
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final char[] results = new char[list.size()];
             for (int i = 0; i < results.length; i++) {
                 results[i] = ((String) list.get(i)).charAt(0);

diff --git a/src/main/java/org/apache/commons/beanutils/converters/DoubleArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/DoubleArrayConverter.java
@@ -122,7 +122,7 @@ public Object convert(final Class type, final Object value) {
         // Parse the input value as a String into elements
         // and convert to the appropriate type
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final double[] results = new double[list.size()];
             for (int i = 0; i < results.length; i++) {
                 results[i] = Double.parseDouble((String) list.get(i));

diff --git a/src/main/java/org/apache/commons/beanutils/converters/FloatArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/FloatArrayConverter.java
@@ -122,7 +122,7 @@ public Object convert(final Class type, final Object value) {
         // Parse the input value as a String into elements
         // and convert to the appropriate type
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final float[] results = new float[list.size()];
             for (int i = 0; i < results.length; i++) {
                 results[i] = Float.parseFloat((String) list.get(i));

diff --git a/src/main/java/org/apache/commons/beanutils/converters/IntegerArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/IntegerArrayConverter.java
@@ -122,7 +122,7 @@ public Object convert(final Class type, final Object value) {
         // Parse the input value as a String into elements
         // and convert to the appropriate type
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final int[] results = new int[list.size()];
             for (int i = 0; i < results.length; i++) {
                 results[i] = Integer.parseInt((String) list.get(i));

diff --git a/src/main/java/org/apache/commons/beanutils/converters/LongArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/LongArrayConverter.java
@@ -122,7 +122,7 @@ public Object convert(final Class type, final Object value) {
         // Parse the input value as a String into elements
         // and convert to the appropriate type
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final long[] results = new long[list.size()];
             for (int i = 0; i < results.length; i++) {
                 results[i] = Long.parseLong((String) list.get(i));

diff --git a/src/main/java/org/apache/commons/beanutils/converters/ShortArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/ShortArrayConverter.java
@@ -122,7 +122,7 @@ public Object convert(final Class type, final Object value) {
         // Parse the input value as a String into elements
         // and convert to the appropriate type
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final short[] results = new short[list.size()];
             for (int i = 0; i < results.length; i++) {
                 results[i] = Short.parseShort((String) list.get(i));

diff --git a/src/main/java/org/apache/commons/beanutils/converters/StringArrayConverter.java b/src/main/java/org/apache/commons/beanutils/converters/StringArrayConverter.java
@@ -148,7 +148,7 @@ public Object convert(final Class type, final Object value) {
         // Parse the input value as a String into elements
         // and convert to the appropriate type
         try {
-            final List list = parseElements(value.toString());
+            final List<String> list = parseElements(value.toString());
             final String[] results = new String[list.size()];
             for (int i = 0; i < results.length; i++) {
                 results[i] = (String) list.get(i);