Author: toad
Date: 2008-03-01 21:32:31 +0000 (Sat, 01 Mar 2008)
New Revision: 18304

Modified:
   trunk/freenet/src/freenet/node/FNPPacketMangler.java
   trunk/freenet/src/freenet/node/KeyTracker.java
   trunk/freenet/src/freenet/node/PeerNode.java
Log:
PeerNode: Make pingAverage private.

Modified: trunk/freenet/src/freenet/node/FNPPacketMangler.java
===================================================================
--- trunk/freenet/src/freenet/node/FNPPacketMangler.java        2008-03-01 
21:17:08 UTC (rev 18303)
+++ trunk/freenet/src/freenet/node/FNPPacketMangler.java        2008-03-01 
21:32:31 UTC (rev 18304)
@@ -1926,7 +1926,7 @@

                if((seqNumber != -1) && tracker.alreadyReceived(seqNumber)) {
                        tracker.queueAck(seqNumber); // Must keep the 
connection open!
-                       if(logMINOR) Logger.minor(this, "Received packet twice 
("+seqNumber+") from "+tracker.pn.getPeer()+": "+seqNumber+" 
("+TimeUtil.formatTime((long) tracker.pn.pingAverage.currentValue(), 2, true)+" 
ping avg)");
+                       if(logMINOR) Logger.minor(this, "Received packet twice 
("+seqNumber+") from "+tracker.pn.getPeer()+": "+seqNumber+" 
("+TimeUtil.formatTime((long) tracker.pn.averagePingTime(), 2, true)+" ping 
avg)");
                        return;
                }


Modified: trunk/freenet/src/freenet/node/KeyTracker.java
===================================================================
--- trunk/freenet/src/freenet/node/KeyTracker.java      2008-03-01 21:17:08 UTC 
(rev 18303)
+++ trunk/freenet/src/freenet/node/KeyTracker.java      2008-03-01 21:32:31 UTC 
(rev 18304)
@@ -425,8 +425,8 @@
          */
                public void onAcked() {
                        long t = Math.max(0, System.currentTimeMillis() - 
createdTime);
-                       pn.pingAverage.report(t);
-                       if(logMINOR) Logger.minor(this, "Reported round-trip 
time of "+TimeUtil.formatTime(t, 2, true)+" on "+pn.getPeer()+" (avg 
"+TimeUtil.formatTime((long)pn.pingAverage.currentValue(),2,true)+", 
#"+packetNumber+ ')');
+                       pn.reportPing(t);
+                       if(logMINOR) Logger.minor(this, "Reported round-trip 
time of "+TimeUtil.formatTime(t, 2, true)+" on "+pn.getPeer()+" (avg 
"+TimeUtil.formatTime((long)pn.averagePingTime(),2,true)+", #"+packetNumber+ 
')');
                }

                long urgentDelay() {
@@ -646,7 +646,7 @@
         QueuedAckRequest qr = 
(QueuedAckRequest)ackRequestQueue.removeByKey(new Integer(seqNo));
        if(qr != null) qr.onAcked();
        else
-               Logger.normal(this, "Removing ack request twice? Null on 
"+seqNo+" from "+pn.getPeer()+" ("+TimeUtil.formatTime((int) 
pn.pingAverage.currentValue(), 2, true)+" ping avg)");
+               Logger.normal(this, "Removing ack request twice? Null on 
"+seqNo+" from "+pn.getPeer()+" ("+TimeUtil.formatTime((int) 
pn.averagePingTime(), 2, true)+" ping avg)");
     }

     /**

Modified: trunk/freenet/src/freenet/node/PeerNode.java
===================================================================
--- trunk/freenet/src/freenet/node/PeerNode.java        2008-03-01 21:17:08 UTC 
(rev 18303)
+++ trunk/freenet/src/freenet/node/PeerNode.java        2008-03-01 21:32:31 UTC 
(rev 18304)
@@ -2617,7 +2617,7 @@
        final static int MAX_PINGS = 5;
        final LRUHashtable pingsSentTimes = new LRUHashtable();
        long pingNumber;
-       final RunningAverage pingAverage;
+       private final RunningAverage pingAverage;

        /**
         * @return The probability of a request sent to this peer being 
rejected (locally)
@@ -3608,4 +3608,8 @@
                if(peer == null) return false; // presumably
                return !IPUtil.isValidAddress(peer.getAddress(), false);
        }
+
+       public void reportPing(long t) {
+               this.pingAverage.report(t);
+       }
 }


Reply via email to