Author: toad
Date: 2009-03-12 21:30:48 +0000 (Thu, 12 Mar 2009)
New Revision: 26012

Modified:
   branches/db4o/freenet/src/freenet/node/fcp/ClientPutDir.java
Log:
Doh, voodoo didn't fix it


Modified: branches/db4o/freenet/src/freenet/node/fcp/ClientPutDir.java
===================================================================
--- branches/db4o/freenet/src/freenet/node/fcp/ClientPutDir.java        
2009-03-12 21:29:00 UTC (rev 26011)
+++ branches/db4o/freenet/src/freenet/node/fcp/ClientPutDir.java        
2009-03-12 21:30:48 UTC (rev 26012)
@@ -413,24 +413,10 @@
        public void onSuccess(FetchResult result, ClientGetter state, 
ObjectContainer container) {}
        
        public void onSuccess(BaseClientPutter state, ObjectContainer 
container) {
-               /** FIXME: EVIL BAD VOODOO! POSSIBLE DB4O BUG!
-                * Without this line, manifestElements is stored correctly, is 
never updated,
-                * yet returns 0 elements in freeData() and thus leaks the 
ManifestElement and
-                * the bucket inside it. AFAICS this is a db4o bug, as 
objectCanUpdate() is 
-                * never called for the HashMap. Also it goes away when you 
attach a debugger.
-                * :< */
-               container.activate(manifestElements, 2);
                super.onSuccess(state, container);
        }
        
        public void onFailure(InsertException e, BaseClientPutter state, 
ObjectContainer container) {
-               /** FIXME: EVIL BAD VOODOO! POSSIBLE DB4O BUG!
-                * Without this line, manifestElements is stored correctly, is 
never updated,
-                * yet returns 0 elements in freeData() and thus leaks the 
ManifestElement and
-                * the bucket inside it. AFAICS this is a db4o bug, as 
objectCanUpdate() is 
-                * never called for the HashMap. Also it goes away when you 
attach a debugger.
-                * :< */
-               container.activate(manifestElements, 2);
                super.onFailure(e, state, container);
        }
 

_______________________________________________
cvs mailing list
[email protected]
http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs

Reply via email to