You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by tn...@apache.org on 2012/06/23 20:29:45 UTC

svn commit: r1353172 - /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/comparators/TransformingComparator.java

Author: tn
Date: Sat Jun 23 18:29:44 2012
New Revision: 1353172

URL: http://svn.apache.org/viewvc?rev=1353172&view=rev
Log:
[COLLECTIONS-324] Made protected fields final to improve thread-safety. Thanks to sebb for the report.

Modified:
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/comparators/TransformingComparator.java

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/comparators/TransformingComparator.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/comparators/TransformingComparator.java?rev=1353172&r1=1353171&r2=1353172&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/comparators/TransformingComparator.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/comparators/TransformingComparator.java Sat Jun 23 18:29:44 2012
@@ -35,9 +35,9 @@ import org.apache.commons.collections.Tr
 public class TransformingComparator<E> implements Comparator<E> {
     
     /** The decorated comparator. */
-    protected Comparator<E> decorated;
+    protected final Comparator<E> decorated;
     /** The transformer being used. */    
-    protected Transformer<? super E, ? extends E> transformer;
+    protected final Transformer<? super E, ? extends E> transformer;
 
     //-----------------------------------------------------------------------
     /**