Clean up SSHTransportStack a bit 82/108382/1
authorRobert Varga <robert.varga@pantheon.tech>
Fri, 13 Oct 2023 08:28:54 +0000 (10:28 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 13 Oct 2023 08:29:09 +0000 (10:29 +0200)
Shuffle justification and fields a bit.

JIRA: NETCONF-1106
Change-Id: I5e1f9a0efcc7c50871042045147688b66e5cf3c6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
transport/transport-ssh/src/main/java/org/opendaylight/netconf/transport/ssh/SSHTransportStack.java

index a9f0ffcdf87c3a783c3cee2016c2cdf1d3ef4a11..587425fbe82495497d72787c031ceaa943b336df 100644 (file)
@@ -29,15 +29,13 @@ import org.opendaylight.netconf.transport.api.TransportStack;
  */
 public abstract sealed class SSHTransportStack extends AbstractOverlayTransportStack<SSHTransportChannel>
         implements SessionListener permits SSHClient, SSHServer {
-    private final TransportIoService ioService;
-
     private final Map<Long, TransportChannel> unauthUnderlays = new ConcurrentHashMap<>();
     private final Map<Long, Session> sessions = new ConcurrentHashMap<>();
+    private final TransportIoService ioService;
 
-    @SuppressFBWarnings(value = "MC_OVERRIDABLE_METHOD_CALL_IN_CONSTRUCTOR",
-        justification = """
-            SessionListener has default implementations which we do not care about. We have all subclasses in this
-            package and neither of them has additional state""")
+    @SuppressFBWarnings(value = "MC_OVERRIDABLE_METHOD_CALL_IN_CONSTRUCTOR", justification = """
+        SessionListener has default implementations which we do not care about. We have all subclasses in this package
+        and neither of them has additional state""")
     SSHTransportStack(final TransportChannelListener listener, final FactoryManager factoryManager,
             final IoHandler handler) {
         super(listener);