Fix checkstyle 29/107629/4
authorRobert Varga <[email protected]>
Thu, 31 Aug 2023 10:23:46 +0000 (12:23 +0200)
committerRobert Varga <[email protected]>
Thu, 31 Aug 2023 11:44:12 +0000 (13:44 +0200)
Upgraded checkstyle is flushing out new things, fix them up.

Change-Id: I46ac6cf74c243a80a6866d649914628eb19b3576
Signed-off-by: Robert Varga <[email protected]>
apps/callhome-provider/src/main/java/org/opendaylight/netconf/callhome/mount/CallHomeAuthProviderImpl.java
apps/netconf-topology-singleton/src/test/java/org/opendaylight/netconf/topology/singleton/impl/MountPointEndToEndTest.java
netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/MinaSshNettyChannel.java
netconf/netconf-netty-util/src/test/java/org/opendaylight/netconf/nettyutil/AbstractNetconfSessionNegotiatorTest.java
netconf/tools/netconf-testtool/src/main/java/org/opendaylight/netconf/test/tool/ScaleUtil.java
protocol/netconf-server/src/test/java/org/opendaylight/netconf/server/ConcurrentClientsTest.java

index 1fb765e176e1c11b5e743d45e92b87e15cc55125..e237025bcf55d4fe06bba0ca5d1e5cc574f8d002 100644 (file)
@@ -191,7 +191,7 @@ public class CallHomeAuthProviderImpl implements CallHomeAuthorizationProvider,
         abstract void removeDevice(String publicKey, Device device);
     }
 
-    private static class DeviceConfig extends AbstractDeviceListener {
+    private static final class DeviceConfig extends AbstractDeviceListener {
         private final ConcurrentMap<PublicKey, Device> byPublicKey = new ConcurrentHashMap<>();
         private final AuthorizedKeysDecoder keyDecoder = new AuthorizedKeysDecoder();
 
@@ -225,7 +225,7 @@ public class CallHomeAuthProviderImpl implements CallHomeAuthorizationProvider,
         }
     }
 
-    private static class DeviceOp extends AbstractDeviceListener {
+    private static final class DeviceOp extends AbstractDeviceListener {
         private final ConcurrentMap<String, Device> byPublicKey = new ConcurrentHashMap<>();
 
         Device get(final PublicKey serverKey) {
@@ -251,7 +251,7 @@ public class CallHomeAuthProviderImpl implements CallHomeAuthorizationProvider,
         }
     }
 
-    private static class GlobalConfig implements DataTreeChangeListener<Global> {
+    private static final class GlobalConfig implements DataTreeChangeListener<Global> {
         private volatile Global current = null;
 
         @Override
index 0519f9b99a3fc5ee1b71968a92d75f2edde12701..ac3186200e8fc12ae9db32e95cb185c0f7cf3997 100644 (file)
@@ -723,7 +723,7 @@ public class MountPointEndToEndTest extends AbstractBaseSchemasTest {
         return rpcDefinition;
     }
 
-    private static class TopDOMRpcImplementation implements DOMRpcImplementation {
+    private static final class TopDOMRpcImplementation implements DOMRpcImplementation {
         private volatile SettableFuture<Entry<DOMRpcIdentifier, NormalizedNode>> rpcInvokedFuture;
         private volatile ListenableFuture<DOMRpcResult> returnFuture;
 
index 94807409df04c4fa9a162ac292c12094e7c571b8..8a65eff04b802a961b33ff4d4c36277fafaa5330 100644 (file)
@@ -131,7 +131,7 @@ class MinaSshNettyChannel extends AbstractServerChannel {
         throw new IllegalStateException("Outbound writes to SSH should be done by SSH Write handler");
     }
 
-    private class SshUnsafe extends AbstractUnsafe {
+    private final class SshUnsafe extends AbstractUnsafe {
         @Override
         public void connect(final SocketAddress remoteAddress, final SocketAddress localAddress,
                 final ChannelPromise promise) {
index b6870c3f636c65890145f2d18df26a6deda80b35..7f43857b38313805894ad604dbc4a04892432093 100644 (file)
@@ -184,7 +184,7 @@ public class AbstractNetconfSessionNegotiatorTest {
         doReturn(timeout).when(timer).newTimeout(any(), eq(100L), eq(TimeUnit.MILLISECONDS));
     }
 
-    private static class CloseDetector extends ChannelOutboundHandlerAdapter {
+    private static final class CloseDetector extends ChannelOutboundHandlerAdapter {
         @Override
         public void close(final ChannelHandlerContext ctx, final ChannelPromise promise) {
             // Override needed so @Skip from superclass is not effective
index 186251a19c552e218cb75c27b66552c31d6169bf..fbc1285eb3f1d8b16dc70ebc0c6cba33fa4a7230 100644 (file)
@@ -294,7 +294,7 @@ public final class ScaleUtil {
         }
     }
 
-    private static class TimeoutGuard implements Callable<Void> {
+    private static final class TimeoutGuard implements Callable<Void> {
         @Override
         public Void call() {
             resultsLog.warn("Timeout for scale test reached after: {} ..aborting", STOPWATCH);
index dd1dee6991c4ebaec6c67207919efadd91b34025..701bd82436c41810b035367acdd134e53098a565 100644 (file)
@@ -230,8 +230,7 @@ public class ConcurrentClientsTest {
     /**
      * Responds to all operations except start-exi and counts all requests.
      */
-    private static class TestingNetconfOperation implements NetconfOperation {
-
+    private static final class TestingNetconfOperation implements NetconfOperation {
         private final AtomicLong counter = new AtomicLong();
 
         @Override
@@ -262,7 +261,7 @@ public class ConcurrentClientsTest {
     /**
      * Hardcoded operation service factory.
      */
-    private static class TestingOperationServiceFactory implements NetconfOperationServiceFactory {
+    private static final class TestingOperationServiceFactory implements NetconfOperationServiceFactory {
         private final NetconfOperation[] operations;
 
         TestingOperationServiceFactory(final NetconfOperation... operations) {