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 2013/05/04 17:08:51 UTC

svn commit: r1479122 - /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java

Author: tn
Date: Sat May  4 15:08:41 2013
New Revision: 1479122

URL: http://svn.apache.org/r1479122
Log:
Removed unused field, the parent bidi map is accessed via decorated().

Modified:
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java?rev=1479122&r1=1479121&r2=1479122&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap.java Sat May  4 15:08:41 2013
@@ -224,9 +224,6 @@ public class DualTreeBidiMap<K, V> exten
      * Internal sorted map view.
      */
     protected static class ViewMap<K, V> extends AbstractSortedMapDecorator<K, V> {
-        /** The parent bidi map. */
-        private final DualTreeBidiMap<K, V> bidi; // TODO not actually used - WHY?
-
         /**
          * Constructor.
          * @param bidi  the parent bidi map
@@ -237,7 +234,6 @@ public class DualTreeBidiMap<K, V> exten
             // use the normalMap as the filtered map, but reverseMap as the full map
             // this forces containsValue and clear to be overridden
             super(new DualTreeBidiMap<K, V>(sm, bidi.reverseMap, bidi.inverseBidiMap));
-            this.bidi = decorated();
         }
 
         @Override