fixed bug in JAMailboxFactory.removeClosable

laforge49 · Jan 3, 2013 · 4be5412 · 4be5412
1 parent 7481bd5
commit 4be5412
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/pom.xml b/pom.xml
@@ -5,7 +5,7 @@
 
     <groupId>org.agilewiki.jactor</groupId>
     <artifactId>jactor</artifactId>
-    <version>4.6.0</version>
+    <version>4.6.1.0</version>
     <packaging>jar</packaging>
     <name>JActor</name>
     <description>Lock-free Java Actors</description>

diff --git a/release.xml b/release.xml
@@ -4,7 +4,7 @@
 
     <groupId>org.agilewiki.jactor</groupId>
     <artifactId>jactor</artifactId>
-    <version>4.6.1-SNAPSHOT</version>
+    <version>4.6.1.0-SNAPSHOT</version>
     <packaging>jar</packaging>
     <name>JActor</name>
     <description>Lock-free Java Actors</description>

diff --git a/src/main/java/org/agilewiki/jactor/JAMailboxFactory.java b/src/main/java/org/agilewiki/jactor/JAMailboxFactory.java
@@ -89,12 +89,14 @@ public ThreadManager getThreadManager() {
 
     @Override
     public boolean addClosable(Closable closable) {
+        if (closable == null)
+            throw new IllegalArgumentException("may not be null");
         return closables.add(closable);
     }
 
     @Override
     public boolean removeClosable(Closable closable) {
-        return closables.add(closable);
+        return closables.remove(closable);
     }
 
     /**