[freenet-cvs] r15301 - trunk/freenet/src/freenet/node

[t...@freenetproject.org]

Author: toad
Date: 2007-09-25 16:19:53 +0000 (Tue, 25 Sep 2007)
New Revision: 15301

Modified:
   trunk/freenet/src/freenet/node/OpennetManager.java
Log:
Fix bug I introduced a few revisions ago: We were never dumping old nodes!

Modified: trunk/freenet/src/freenet/node/OpennetManager.java
===================================================================
--- trunk/freenet/src/freenet/node/OpennetManager.java  2007-09-25 16:05:01 UTC 
(rev 15300)
+++ trunk/freenet/src/freenet/node/OpennetManager.java  2007-09-25 16:19:53 UTC 
(rev 15301)
@@ -239,6 +239,7 @@
         * @return True if the node was added / should be added.
         */
        public boolean wantPeer(PeerNode nodeToAddNow, boolean addAtLRU) {
+               boolean notMany = false;
                boolean ret = true;
                boolean noDisconnect;
                synchronized(this) {
@@ -260,11 +261,11 @@
                                        if(logMINOR) Logger.minor(this, "Want 
peer because not enough opennet nodes");
                                }
                                timeLastOffered = System.currentTimeMillis();
-                               ret = true;
+                               notMany = true;
                        }
                        noDisconnect = successCount < 
MIN_SUCCESS_BETWEEN_DROP_CONNS;
                }
-               if(ret) {
+               if(notMany) {
                        if(nodeToAddNow != null)
                                node.peers.addPeer(nodeToAddNow, true); // Add 
to peers outside the OM lock
                        return true;