From 7028d9f1483d6f1e77bb0f5ebd0ecc6239e431c5 Mon Sep 17 00:00:00 2001
From: Jean-Noël Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Mon, 14 Nov 2016 15:38:45 +0000
Subject: [PATCH] AutoRefactor'ed use Map.entrySet() instead of Map.keySet() in a loop

---
 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/LDAPConnectionPool.java |    9 +++++----
 1 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/LDAPConnectionPool.java b/opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/LDAPConnectionPool.java
index d31fa1a..1a4e5db 100644
--- a/opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/LDAPConnectionPool.java
+++ b/opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/LDAPConnectionPool.java
@@ -20,6 +20,7 @@
 
 import java.util.ArrayList;
 import java.util.HashMap;
+import java.util.Map;
 
 import javax.net.ssl.KeyManager;
 
@@ -85,9 +86,8 @@
    *         {@code false} otherwise.
    */
   public boolean isConnectionRegistered(ConnectionWrapper conn) {
-    for (String key : connectionTable.keySet())
+    for (ConnectionRecord cr : connectionTable.values())
     {
-      final ConnectionRecord cr = connectionTable.get(key);
       if (cr.conn != null)
       {
         final ConnectionWrapper c = cr.conn.getConnectionWrapper();
@@ -245,8 +245,9 @@
     String targetKey = null;
     ConnectionRecord targetRecord = null;
     synchronized(this) {
-      for (String key : connectionTable.keySet()) {
-        ConnectionRecord cr = connectionTable.get(key);
+      for (Map.Entry<String, ConnectionRecord> entry : connectionTable.entrySet()) {
+        String key = entry.getKey();
+        ConnectionRecord cr = entry.getValue();
         if (cr.conn == conn) {
           targetKey = key;
           targetRecord = cr;

--
Gitblit v1.10.0