Author: toad
Date: 2009-04-03 00:25:50 +0000 (Fri, 03 Apr 2009)
New Revision: 26402
Modified:
trunk/freenet/src/freenet/client/async/DatastoreChecker.java
Log:
Simplify, avoid minor leak, logging
Modified: trunk/freenet/src/freenet/client/async/DatastoreChecker.java
===================================================================
--- trunk/freenet/src/freenet/client/async/DatastoreChecker.java
2009-04-03 00:20:50 UTC (rev 26401)
+++ trunk/freenet/src/freenet/client/async/DatastoreChecker.java
2009-04-03 00:25:50 UTC (rev 26402)
@@ -99,7 +99,7 @@
totalSize +=
persistentKeys[i].get(j).length;
}
if(totalSize > MAX_PERSISTENT_KEYS) {
- if(logMINOR) Logger.minor(this, "Persistent
datastore checker queue alreadyfull");
+ if(logMINOR) Logger.minor(this, "Persistent
datastore checker queue already full");
return;
}
}
@@ -113,14 +113,12 @@
for(DatastoreCheckerItem item : results) {
if(item.chosenBy == context.bootID) continue;
SendableGet getter = item.getter;
- if(getter == null) {
- // FIXME is this normal or isn't it?
Probably ... if we don't always delete the DCI's ...
+ if(getter == null ||
!container.ext().isStored(getter)) {
if(logMINOR) Logger.minor(this,
"Ignoring DatastoreCheckerItem because the SendableGet has already been deleted
from the database");
container.delete(item);
continue;
}
BlockSet blocks = item.blocks;
- if(!container.ext().isStored(getter)) continue;
// Already deleted
container.activate(getter, 1);
boolean dontCache = getter.dontCache(container);
ClientRequestScheduler sched =
getter.getScheduler(context);
_______________________________________________
cvs mailing list
[email protected]
http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs