From 83f86f11c27378d650c3bfc7e55397687062edbc Mon Sep 17 00:00:00 2001
From: Jean-Noël Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Fri, 22 Apr 2016 14:23:28 +0000
Subject: [PATCH] Improvements suggested by UCDetector: remove dead code, add final keywords, change visibilities

---
 opendj-server-legacy/src/main/java/org/opends/admin/ads/util/TrustedSocketFactory.java |   25 +++++++++++--------------
 1 files changed, 11 insertions(+), 14 deletions(-)

diff --git a/opendj-server-legacy/src/main/java/org/opends/admin/ads/util/TrustedSocketFactory.java b/opendj-server-legacy/src/main/java/org/opends/admin/ads/util/TrustedSocketFactory.java
index b7723ba..f404a36 100644
--- a/opendj-server-legacy/src/main/java/org/opends/admin/ads/util/TrustedSocketFactory.java
+++ b/opendj-server-legacy/src/main/java/org/opends/admin/ads/util/TrustedSocketFactory.java
@@ -31,24 +31,24 @@
 import javax.net.ssl.TrustManager;
 
 /** An implementation of SSLSocketFactory. */
-public class TrustedSocketFactory extends SSLSocketFactory
+class TrustedSocketFactory extends SSLSocketFactory
 {
-  private static Map<Thread, TrustManager> hmTrustManager = new HashMap<>();
-  private static Map<Thread, KeyManager> hmKeyManager = new HashMap<>();
+  private static final Map<Thread, TrustManager> hmTrustManager = new HashMap<>();
+  private static final Map<Thread, KeyManager> hmKeyManager = new HashMap<>();
 
-  private static Map<TrustManager, SocketFactory> hmDefaultFactoryTm = new HashMap<>();
-  private static Map<KeyManager, SocketFactory> hmDefaultFactoryKm = new HashMap<>();
+  private static final Map<TrustManager, SocketFactory> hmDefaultFactoryTm = new HashMap<>();
+  private static final Map<KeyManager, SocketFactory> hmDefaultFactoryKm = new HashMap<>();
 
   private SSLSocketFactory innerFactory;
-  private TrustManager trustManager;
-  private KeyManager   keyManager;
+  private final TrustManager trustManager;
+  private final KeyManager keyManager;
 
   /**
    * Constructor of the TrustedSocketFactory.
    * @param trustManager the trust manager to use.
    * @param keyManager   the key manager to use.
    */
-  public TrustedSocketFactory(TrustManager trustManager, KeyManager keyManager)
+  TrustedSocketFactory(TrustManager trustManager, KeyManager keyManager)
   {
     this.trustManager = trustManager;
     this.keyManager   = keyManager;
@@ -63,8 +63,7 @@
    * @param keyManager
    *          the key manager to use.
    */
-  public static synchronized void setCurrentThreadTrustManager(
-      TrustManager trustManager, KeyManager keyManager)
+  static synchronized void setCurrentThreadTrustManager(TrustManager trustManager, KeyManager keyManager)
   {
     setThreadTrustManager(trustManager, Thread.currentThread());
     setThreadKeyManager  (keyManager, Thread.currentThread());
@@ -75,8 +74,7 @@
    * @param trustManager the trust manager to use.
    * @param thread the thread where we want to use the provided trust manager.
    */
-  public static synchronized void setThreadTrustManager(
-      TrustManager trustManager, Thread thread)
+  static synchronized void setThreadTrustManager(TrustManager trustManager, Thread thread)
   {
     TrustManager currentTrustManager = hmTrustManager.get(thread);
     if (currentTrustManager != null) {
@@ -93,8 +91,7 @@
    * @param keyManager the key manager to use.
    * @param thread the thread where we want to use the provided key manager.
    */
-  public static synchronized void setThreadKeyManager(
-      KeyManager keyManager, Thread thread)
+  static synchronized void setThreadKeyManager(KeyManager keyManager, Thread thread)
   {
     KeyManager currentKeyManager = hmKeyManager.get(thread);
     if (currentKeyManager != null) {

--
Gitblit v1.10.0