From 771bcd1a76b7f40f2338c047eced3e34794c33a1 Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Wed, 30 Aug 2023 16:50:22 +0200 Subject: [PATCH] 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 --- .../config/threadpool/util/FlexibleThreadPoolWrapper.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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; -- 2.36.6