* Florent Daignière <[EMAIL PROTECTED]> [2007-12-29 12:05:17]:
> * [EMAIL PROTECTED] <[EMAIL PROTECTED]> [2007-12-29 01:46:31]:
>
> > Author: robert
> > Date: 2007-12-29 01:46:31 +0000 (Sat, 29 Dec 2007)
> > New Revision: 16834
> >
> > Modified:
> > trunk/freenet/src/freenet/node/PeerNode.java
> > Log:
> > maybe help wont-fetch-ark deadlock #2
> >
> >
> > Modified: trunk/freenet/src/freenet/node/PeerNode.java
> > ===================================================================
> > --- trunk/freenet/src/freenet/node/PeerNode.java 2007-12-29 01:41:06 UTC
> > (rev 16833)
> > +++ trunk/freenet/src/freenet/node/PeerNode.java 2007-12-29 01:46:31 UTC
> > (rev 16834)
> > @@ -2782,7 +2782,7 @@
> >
> > synchronized void updateShouldDisconnectNow() {
> > //FIXME: We should not update VERIFIED unless we HANDSHAKE WITH
> > THE NODE
> > - if (isConnected()) {
> > + if (isConnected() || verifiedIncompatibleOlderVersion ||
> > verifiedIncompatibleNewerVersion) {
> > verifiedIncompatibleOlderVersion =
> > forwardInvalidVersion();
> > verifiedIncompatibleNewerVersion =
> > reverseInvalidVersion();
> > }
>
> I suggest you call isRoutable() instead
>
> NextGen$Hmm, on a latter thought, I don't understand why it helps... nor why we are calling it from PacketSender What about getting rid of both the "if" branch and the call in PacketSender ? NextGen$
signature.asc
Description: Digital signature
_______________________________________________ Devl mailing list [email protected] http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl
