On Friday 03 April 2009 09:50:34 Daniel Cheng wrote:
> On Fri, Apr 3, 2009 at 7:59 AM,  <[email protected]> wrote:
> > Author: toad
> > Date: 2009-04-02 23:59:42 +0000 (Thu, 02 Apr 2009)
> > New Revision: 26397
> >
> > Modified:
> >   trunk/freenet/src/freenet/client/async/ClientRequestSchedulerCore.java
> > Log:
> > Better priority/retrycount comparison
> >
> >
> > Modified: 
trunk/freenet/src/freenet/client/async/ClientRequestSchedulerCore.java
> > ===================================================================
> > --- 
trunk/freenet/src/freenet/client/async/ClientRequestSchedulerCore.java      
2009-04-02 
23:54:43 UTC (rev 26396)
> > +++ 
trunk/freenet/src/freenet/client/async/ClientRequestSchedulerCore.java      
2009-04-02 
23:59:42 UTC (rev 26397)
> > @@ -508,8 +508,10 @@
> 
>                                                        Logger.minor(this, 
> "Ignoring 
cancelled recently succeeded item "+altReq);
> >                                                altReq = null;
> >                                        }
> > -                                       if(altReq != null && 
altReq.getPriorityClass(container) <= choosenPriorityClass &&
> > -                                                       
fixRetryCount(altReq.getRetryCount()) <= chosenTracker.getNumber() 
&& !altReq.isEmpty(container) && altReq != req) {
> > +                                       int prio = 
altReq.getPriorityClass(container);
> 
> altReq maybe null.
> This is throwing NPE

Doh, thanks for fixing!
> 
> > +                                       if(altReq != null &&

Attachment: signature.asc
Description: This is a digitally signed message part.

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

Reply via email to