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/18 21:22:14 UTC

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

Author: sebb
Date: Mon Oct 18 19:22:14 2010
New Revision: 1023961

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

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

Modified: commons/proper/collections/trunk/src/java/org/apache/commons/collections/bidimap/DualTreeBidiMap.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/java/org/apache/commons/collections/bidimap/DualTreeBidiMap.java?rev=1023961&r1=1023960&r2=1023961&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/java/org/apache/commons/collections/bidimap/DualTreeBidiMap.java (original)
+++ commons/proper/collections/trunk/src/java/org/apache/commons/collections/bidimap/DualTreeBidiMap.java Mon Oct 18 19:22:14 2010
@@ -383,12 +383,12 @@ public class DualTreeBidiMap<K, V> exten
         out.writeObject(normalMap);
     }
 
-    @SuppressWarnings("unchecked")
     private void readObject(ObjectInputStream in) throws IOException, ClassNotFoundException {
         in.defaultReadObject();
-        normalMap = new TreeMap(comparator);
-        reverseMap = new TreeMap(valueComparator);
-        Map map = (Map) in.readObject();
+        normalMap = new TreeMap<K, V>(comparator);
+        reverseMap = new TreeMap<V, K>(valueComparator);
+        @SuppressWarnings("unchecked") // will fail at runtime if the stream is incorrect
+        Map<K, V> map = (Map<K, V>) in.readObject();
         putAll(map);
     }