[freenet-cvs] r18832 - trunk/freenet/src/freenet/client/async

[t...@freenetproject.org]

Author: toad
Date: 2008-03-29 16:34:31 +0000 (Sat, 29 Mar 2008)
New Revision: 18832

Modified:
   trunk/freenet/src/freenet/client/async/ClientRequestScheduler.java
Log:
removePendingKey(): if there are several SendableGet's for a key, we would 
always delete them all!
IIRC this will happen as a normal part of the changeover from one subsegment to 
another??
Anyway, it's a bad bug!

Modified: trunk/freenet/src/freenet/client/async/ClientRequestScheduler.java
===================================================================
--- trunk/freenet/src/freenet/client/async/ClientRequestScheduler.java  
2008-03-29 16:18:06 UTC (rev 18831)
+++ trunk/freenet/src/freenet/client/async/ClientRequestScheduler.java  
2008-03-29 16:34:31 UTC (rev 18832)
@@ -530,9 +530,9 @@
                                                                
Logger.normal(this, "Not found: "+getter+" for "+key+" removing 
("+gets.length+" getters)");
                                                        return; // not here
                                                }
+                                       }
                                                if(gets[j] == getter || gets[j] 
== null || gets[j].isCancelled()) continue;
                                                newGets[x++] = gets[j];
-                                       }
                                }
                                if(x != gets.length-1) {
                                        SendableGet[] newNewGets = new 
SendableGet[x];