[infinispan-commits] Infinispan SVN: r840 - trunk/cachestore/jdbm/src/main/java/org/infinispan/loaders/jdbm.

[infinispan-commits at lists.jboss.org]

Author: galder.zamarreno at jboss.com
Date: 2009-09-18 12:15:27 -0400 (Fri, 18 Sep 2009)
New Revision: 840

Modified:
   trunk/cachestore/jdbm/src/main/java/org/infinispan/loaders/jdbm/JdbmCacheStore.java
Log:
Always return true since HTree.remove() does not indicate whether the key existed.

Modified: trunk/cachestore/jdbm/src/main/java/org/infinispan/loaders/jdbm/JdbmCacheStore.java
===================================================================
--- trunk/cachestore/jdbm/src/main/java/org/infinispan/loaders/jdbm/JdbmCacheStore.java	2009-09-18 15:33:30 UTC (rev 839)
+++ trunk/cachestore/jdbm/src/main/java/org/infinispan/loaders/jdbm/JdbmCacheStore.java	2009-09-18 16:15:27 UTC (rev 840)
@@ -228,7 +228,8 @@
          log.trace("remove() " + key);
       try {
          tree.remove(key);
-         return true; // TODO the return value is not really important, right?
+         // If the key does not exist, HTree ignores the operation, so always return true
+         return true;
       } catch (IOException e) {
          // can happen during normal operation
          return false;