From: Robert Varga Date: Wed, 30 Aug 2023 14:50:22 +0000 (+0200) Subject: Fix FlexibleThreadPoolWrapper checkstyle X-Git-Tag: v8.0.1~6 X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=commitdiff_plain;h=771bcd1a76b7f40f2338c047eced3e34794c33a1 Fix FlexibleThreadPoolWrapper checkstyle Nested static classes are detected as checkstyle violations if they are not final. Fix that. Change-Id: Ifa5878890cabe845ecf8d98d46f2fc8015a78770 Signed-off-by: Robert Varga --- diff --git a/opendaylight/config/threadpool-config-impl/src/main/java/org/opendaylight/controller/config/threadpool/util/FlexibleThreadPoolWrapper.java b/opendaylight/config/threadpool-config-impl/src/main/java/org/opendaylight/controller/config/threadpool/util/FlexibleThreadPoolWrapper.java index d56765d1bb..6fb1adcb69 100644 --- a/opendaylight/config/threadpool-config-impl/src/main/java/org/opendaylight/controller/config/threadpool/util/FlexibleThreadPoolWrapper.java +++ b/opendaylight/config/threadpool-config-impl/src/main/java/org/opendaylight/controller/config/threadpool/util/FlexibleThreadPoolWrapper.java @@ -102,7 +102,7 @@ public class FlexibleThreadPoolWrapper implements ThreadPool, Closeable { /** * if the max threads are met, then it will raise a rejectedExecution. We then push to the queue. */ - private static class FlexibleRejectionHandler implements RejectedExecutionHandler { + private static final class FlexibleRejectionHandler implements RejectedExecutionHandler { @Override @SuppressWarnings("checkstyle:parameterName") public void rejectedExecution(final Runnable r, final ThreadPoolExecutor executor) { @@ -114,7 +114,7 @@ public class FlexibleThreadPoolWrapper implements ThreadPool, Closeable { } } - private static class ForwardingBlockingQueue + private static final class ForwardingBlockingQueue extends com.google.common.util.concurrent.ForwardingBlockingQueue { private final BlockingQueue delegate;