You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by jb...@apache.org on 2009/04/02 22:14:46 UTC

svn commit: r761412 - /incubator/cassandra/trunk/src/org/apache/cassandra/db/EfficientBidiMap.java

Author: jbellis
Date: Thu Apr  2 20:14:46 2009
New Revision: 761412

URL: http://svn.apache.org/viewvc?rev=761412&view=rev
Log:
put returns previous value, so separate get is redundant

Modified:
    incubator/cassandra/trunk/src/org/apache/cassandra/db/EfficientBidiMap.java

Modified: incubator/cassandra/trunk/src/org/apache/cassandra/db/EfficientBidiMap.java
URL: http://svn.apache.org/viewvc/incubator/cassandra/trunk/src/org/apache/cassandra/db/EfficientBidiMap.java?rev=761412&r1=761411&r2=761412&view=diff
==============================================================================
--- incubator/cassandra/trunk/src/org/apache/cassandra/db/EfficientBidiMap.java (original)
+++ incubator/cassandra/trunk/src/org/apache/cassandra/db/EfficientBidiMap.java Thu Apr  2 20:14:46 2009
@@ -72,10 +72,9 @@
 
     public void put(String key, IColumn column)
     {
-    	IColumn oldColumn = map_.get(key);
-    	if( oldColumn != null )
-    		sortedSet_.remove( oldColumn );
-        map_.put(key, column);
+        IColumn oldColumn = map_.put(key, column);
+        if (oldColumn != null)
+            sortedSet_.remove(oldColumn);
         sortedSet_.add(column);
     }