Cleanup ExtensionSessionManagerImpl

Use a properly-constant singleton and remove unused logger.

Change-Id: I0ed7bc521aaa121a13f07ad8f72b37e00d2b522a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/session/ExtensionSessionManagerImpl.java b/openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/session/ExtensionSessionManagerImpl.java
index af72606c2d8babec38938f4e8f1cd5457f2cda3a..df725ac2af43e219ff057c011a933e1e94c137e0 100644 (file)
--- a/openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/session/ExtensionSessionManagerImpl.java
+++ b/openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/session/ExtensionSessionManagerImpl.java
@@ -9,16 +9,16 @@ package org.opendaylight.openflowplugin.openflow.md.core.session;
 
 import org.opendaylight.openflowplugin.extension.api.core.extension.ExtensionConverterProvider;
 import org.opendaylight.openflowplugin.extension.api.core.session.ExtensionSessionManager;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public final class ExtensionSessionManagerImpl implements ExtensionSessionManager {
-    private static final Logger LOG = LoggerFactory.getLogger(ExtensionSessionManagerImpl.class);
-
-    private static ExtensionSessionManagerImpl INSTANCE = new ExtensionSessionManagerImpl();
+    private static final ExtensionSessionManagerImpl INSTANCE = new ExtensionSessionManagerImpl();
 
     private ExtensionConverterProvider extensionConverterProvider;
 
+    private ExtensionSessionManagerImpl() {
+        // Hidden on purpose
+    }
+
     /**
      * Returns singleton instance.
      */
@@ -26,12 +26,8 @@ public final class ExtensionSessionManagerImpl implements ExtensionSessionManage
         return INSTANCE;
     }
 
-    private ExtensionSessionManagerImpl() {
-    }
-
     @Override
-    public void setExtensionConverterProvider(
-            ExtensionConverterProvider extensionConverterProvider) {
+    public void setExtensionConverterProvider(final ExtensionConverterProvider extensionConverterProvider) {
         this.extensionConverterProvider = extensionConverterProvider;
     }