From de094433da9c78a48dcc5bb18000b45badba732a Mon Sep 17 00:00:00 2001
From: zzz <zzz@mail.i2p>
Date: Fri, 19 Aug 2011 15:29:37 +0000
Subject: [PATCH] cleanup

---
 .../router/networkdb/kademlia/FloodfillMonitorJob.java   | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/router/java/src/net/i2p/router/networkdb/kademlia/FloodfillMonitorJob.java b/router/java/src/net/i2p/router/networkdb/kademlia/FloodfillMonitorJob.java
index 088663fd37..c2c2da2bf4 100644
--- a/router/java/src/net/i2p/router/networkdb/kademlia/FloodfillMonitorJob.java
+++ b/router/java/src/net/i2p/router/networkdb/kademlia/FloodfillMonitorJob.java
@@ -19,8 +19,8 @@ import net.i2p.util.Log;
  *
  */
 class FloodfillMonitorJob extends JobImpl {
-    private Log _log;
-    private FloodfillNetworkDatabaseFacade _facade;
+    private final Log _log;
+    private final FloodfillNetworkDatabaseFacade _facade;
     private long _lastChanged;
     
     private static final int REQUEUE_DELAY = 60*60*1000;
@@ -80,7 +80,7 @@ class FloodfillMonitorJob extends JobImpl {
             return false;
 
         // This list will not include ourselves...
-        List floodfillPeers = _facade.getFloodfillPeers();
+        List<Hash> floodfillPeers = _facade.getFloodfillPeers();
         long now = getContext().clock().now();
         // We know none at all! Must be our turn...
         if (floodfillPeers == null || floodfillPeers.isEmpty()) {
@@ -108,8 +108,7 @@ class FloodfillMonitorJob extends JobImpl {
         int ffcount = floodfillPeers.size();
         int failcount = 0;
         long before = now - 60*60*1000;
-        for (int i = 0; i < ffcount; i++) {
-            Hash peer = (Hash)floodfillPeers.get(i);
+        for (Hash peer : floodfillPeers) {
             PeerProfile profile = getContext().profileOrganizer().getProfile(peer);
             if (profile == null || profile.getLastHeardFrom() < before ||
                 profile.getIsFailing() || getContext().shitlist().isShitlisted(peer) ||
-- 
GitLab