Author: toad
Date: 2009-04-06 15:07:37 +0000 (Mon, 06 Apr 2009)
New Revision: 26563

Modified:
   trunk/freenet/src/freenet/client/async/ClientRequestSchedulerCore.java
Log:
Clarify, thanks sdiz(?) for NPE fix


Modified: trunk/freenet/src/freenet/client/async/ClientRequestSchedulerCore.java
===================================================================
--- trunk/freenet/src/freenet/client/async/ClientRequestSchedulerCore.java      
2009-04-06 14:41:13 UTC (rev 26562)
+++ trunk/freenet/src/freenet/client/async/ClientRequestSchedulerCore.java      
2009-04-06 15:07:37 UTC (rev 26563)
@@ -510,8 +510,7 @@
                                        }
                                        if (altReq != null) {
                                        int prio = 
altReq.getPriorityClass(container);
-                                       if( 
-                                                       (prio < 
choosenPriorityClass || (prio == choosenPriorityClass && 
fixRetryCount(altReq.getRetryCount()) <= chosenTracker.getNumber()))
+                                       if((prio < choosenPriorityClass || 
(prio == choosenPriorityClass && fixRetryCount(altReq.getRetryCount()) <= 
chosenTracker.getNumber()))
                                                                        && 
!altReq.isEmpty(container) && altReq != req) {
                                                // Use the recent one instead
                                                if(logMINOR)
@@ -538,11 +537,10 @@
                                                if(logMINOR)
                                                        Logger.minor(this, 
"Maybe using recently succeeded item from "+altRGA);
                                                SendableRequest altReq = 
(SendableRequest) altRGA.removeRandom(starter, container, context);
-                                               container.activate(altReq, 1);
                                                if(altReq != null) {
+                                                       
container.activate(altReq, 1);
                                                        int prio = 
altReq.getPriorityClass(container);
-                                                       if(altReq != null && 
-                                                                       (prio < 
choosenPriorityClass || (prio == choosenPriorityClass && 
fixRetryCount(altReq.getRetryCount()) <= chosenTracker.getNumber()))
+                                                       if((prio < 
choosenPriorityClass || (prio == choosenPriorityClass && 
fixRetryCount(altReq.getRetryCount()) <= chosenTracker.getNumber()))
                                                                                
        && !altReq.isEmpty(container) && altReq != req) {
                                                                // Use the 
recent one instead
                                                                if(logMINOR)

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

Reply via email to