You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2010/10/19 05:16:17 UTC

svn commit: r1024100 - /commons/proper/collections/trunk/src/java/org/apache/commons/collections/list/TransformedList.java

Author: sebb
Date: Tue Oct 19 03:16:15 2010
New Revision: 1024100

URL: http://svn.apache.org/viewvc?rev=1024100&view=rev
Log:
Generics

Modified:
    commons/proper/collections/trunk/src/java/org/apache/commons/collections/list/TransformedList.java

Modified: commons/proper/collections/trunk/src/java/org/apache/commons/collections/list/TransformedList.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/java/org/apache/commons/collections/list/TransformedList.java?rev=1024100&r1=1024099&r2=1024100&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/java/org/apache/commons/collections/list/TransformedList.java (original)
+++ commons/proper/collections/trunk/src/java/org/apache/commons/collections/list/TransformedList.java Tue Oct 19 03:16:15 2010
@@ -73,11 +73,11 @@ public class TransformedList<E> extends 
      * @throws IllegalArgumentException if list or transformer is null
      * @since Commons Collections 3.3
      */
-    // TODO: Generics
-    public static List decorateTransform(List list, Transformer transformer) {
-        TransformedList decorated = new TransformedList(list, transformer);
+    public static <E> List<E> decorateTransform(List<E> list, Transformer<? super E, ? extends E> transformer) {
+        TransformedList<E> decorated = new TransformedList<E>(list, transformer);
         if (transformer != null && list != null && list.size() > 0) {
-            Object[] values = list.toArray();
+            @SuppressWarnings("unchecked") // list is of type E
+            E[] values = (E[]) list.toArray();
             list.clear();
             for(int i=0; i<values.length; i++) {
                 decorated.decorated().add(transformer.transform(values[i]));