You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by tr...@apache.org on 2007/08/17 14:25:47 UTC

svn commit: r567041 - /mina/trunk/core/src/main/java/org/apache/mina/util/ConcurrentHashSet.java

Author: trustin
Date: Fri Aug 17 05:25:46 2007
New Revision: 567041

URL: http://svn.apache.org/viewvc?view=rev&rev=567041
Log:
Fixed NullPointerException in ConcurrentHashSet

Modified:
    mina/trunk/core/src/main/java/org/apache/mina/util/ConcurrentHashSet.java

Modified: mina/trunk/core/src/main/java/org/apache/mina/util/ConcurrentHashSet.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/util/ConcurrentHashSet.java?view=diff&rev=567041&r1=567040&r2=567041
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/util/ConcurrentHashSet.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/util/ConcurrentHashSet.java Fri Aug 17 05:25:46 2007
@@ -41,6 +41,7 @@
 
     @Override
     public boolean add(E o) {
-        return ((ConcurrentMap<E, Boolean>) map).putIfAbsent(o, Boolean.TRUE);
+        Boolean answer = ((ConcurrentMap<E, Boolean>) map).putIfAbsent(o, Boolean.TRUE);
+        return (answer == null);
     }
 }