From c221cfde7a994a8f332b98a7fb58089f6d588388 Mon Sep 17 00:00:00 2001
From: JoelKatz <DavidJoelSchwartz@GMail.com>
Date: Sun, 18 Aug 2013 21:39:46 -0700
Subject: [PATCH] Some filterNodes cleanups.

---
 modules/ripple_app/ledger/ripple_InboundLedger.cpp | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/modules/ripple_app/ledger/ripple_InboundLedger.cpp b/modules/ripple_app/ledger/ripple_InboundLedger.cpp
index 0c5e2d68f2..421a629e82 100644
--- a/modules/ripple_app/ledger/ripple_InboundLedger.cpp
+++ b/modules/ripple_app/ledger/ripple_InboundLedger.cpp
@@ -527,13 +527,15 @@ void InboundLedger::filterNodes (std::vector<SHAMapNode>& nodeIDs, std::vector<u
 
     int dupCount = 0;
 
-    for (unsigned int i = 0; i < nodeIDs.size (); ++i)
+    BOOST_FOREACH(SHAMapNode const& nodeID, nodeIDs)
     {
-        bool isDup = recentNodes.count (nodeIDs[i]) != 0;
-        duplicates.push_back (isDup);
-
-        if (isDup)
+        if (recentNodes.count (nodeID) != 0)
+        {
+            duplicates.push_back (true);
             ++dupCount;
+        }
+        else
+            duplicates.push_back (false);
     }
 
     if (dupCount == nodeIDs.size ())
@@ -543,6 +545,7 @@ void InboundLedger::filterNodes (std::vector<SHAMapNode>& nodeIDs, std::vector<u
         {
             nodeIDs.clear ();
             nodeHashes.clear ();
+            WriteLog (lsTRACE, InboundLedger) << "filterNodes: all are duplicates";
             return;
         }
     }