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

[se...@apache.org]

Author: sebb
Date: Tue Apr 30 09:46:12 2013
New Revision: 1477521

URL: http://svn.apache.org/r1477521
Log:
Privatise fields

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=1477521&r1=1477520&r2=1477521&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 Tue Apr 30 09:46:12 2013
@@ -226,7 +226,7 @@ public class DualTreeBidiMap<K, V> exten
      */
     protected static class ViewMap<K, V> extends AbstractSortedMapDecorator<K, V> {
         /** The parent bidi map. */
-        final DualTreeBidiMap<K, V> bidi;
+        private final DualTreeBidiMap<K, V> bidi; // TODO not actually used - WHY?
 
         /**
          * Constructor.
@@ -294,10 +294,10 @@ public class DualTreeBidiMap<K, V> exten
     protected static class BidiOrderedMapIterator<K, V> implements OrderedMapIterator<K, V>, ResettableIterator<K> {
 
         /** The parent map */
-        protected final AbstractDualBidiMap<K, V> parent;
+        private final AbstractDualBidiMap<K, V> parent;
 
         /** The iterator being decorated */
-        protected ListIterator<Map.Entry<K, V>> iterator;
+        private ListIterator<Map.Entry<K, V>> iterator;
 
         /** The last returned entry */
         private Map.Entry<K, V> last = null;