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:37:19 UTC

svn commit: r1024103 - /commons/proper/collections/trunk/src/java/org/apache/commons/collections/set/TransformedSortedSet.java

Author: sebb
Date: Tue Oct 19 03:37:19 2010
New Revision: 1024103

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

Modified:
    commons/proper/collections/trunk/src/java/org/apache/commons/collections/set/TransformedSortedSet.java

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