On Saturday 29 December 2007 13:34, [EMAIL PROTECTED] wrote: > Author: nextgens > Date: 2007-12-29 13:34:33 +0000 (Sat, 29 Dec 2007) > New Revision: 16839 > > Modified: > trunk/freenet/src/freenet/node/FNPPacketMangler.java > Log: > jfk: Fix the issue spotted by robert (see http://archives.freenetproject.org/message/20071229.105331.b1a491f0.en.html) > > That might solve the "I've lost the exponential" problem
It might? Wouldn't the old code cause an NPE? > > Modified: trunk/freenet/src/freenet/node/FNPPacketMangler.java > =================================================================== > --- trunk/freenet/src/freenet/node/FNPPacketMangler.java 2007-12-29 > 09:36:50 UTC (rev 16838) > +++ trunk/freenet/src/freenet/node/FNPPacketMangler.java 2007-12-29 > 13:34:33 UTC (rev 16839) > @@ -2640,8 +2640,8 @@ > return result; > } > } > - //FIXME: Isn't this wrong? 'result.myExponential' > should be exponential? And what if dhContextToBePrunned is null? > - > if((dhContextToBePrunned.myExponential).equals(result.myExponential)) > + > + if((dhContextToBePrunned != null) && ((dhContextToBePrunned.myExponential).equals(exponential))) > return dhContextToBePrunned; > } > return null; > > _______________________________________________ > cvs mailing list > [EMAIL PROTECTED] > http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs > >
pgpnJJ75UuMoa.pgp
Description: PGP signature
_______________________________________________ Devl mailing list [email protected] http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl
