This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch POOL_2_X
in repository https://gitbox.apache.org/repos/asf/commons-pool.git
The following commit(s) were added to refs/heads/POOL_2_X by this push:
new 34703291 Make private static classes final
34703291 is described below
commit 347032919e2ac01ffd21d537a1336c80733a8989
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Wed Oct 11 10:01:17 2023 -0400
Make private static classes final
---
src/main/java/org/apache/commons/pool2/PoolUtils.java | 2 +-
.../org/apache/commons/pool2/impl/BaseGenericObjectPool.java | 2 +-
src/main/java/org/apache/commons/pool2/impl/EvictionTimer.java | 6 +++---
.../org/apache/commons/pool2/impl/GenericKeyedObjectPool.java | 2 +-
.../apache/commons/pool2/impl/SecurityManagerCallStack.java | 4 ++--
.../java/org/apache/commons/pool2/impl/ThrowableCallStack.java | 2 +-
.../org/apache/commons/pool2/AbstractTestKeyedObjectPool.java | 2 +-
src/test/java/org/apache/commons/pool2/ObjectPoolIssue326.java | 6 +++---
src/test/java/org/apache/commons/pool2/PoolTest.java | 4 ++--
.../apache/commons/pool2/TestBaseKeyedPooledObjectFactory.java | 2 +-
src/test/java/org/apache/commons/pool2/TestBaseObjectPool.java | 2 +-
.../apache/commons/pool2/TestBasePoolableObjectFactory.java | 2 +-
.../commons/pool2/impl/TestAbandonedKeyedObjectPool.java | 2 +-
.../org/apache/commons/pool2/impl/TestAbandonedObjectPool.java | 2 +-
.../apache/commons/pool2/impl/TestGenericKeyedObjectPool.java | 10 +++++-----
.../org/apache/commons/pool2/impl/TestGenericObjectPool.java | 8 ++++----
.../commons/pool2/impl/TestGenericObjectPoolClassLoaders.java | 4 ++--
.../pool2/impl/TestGenericObjectPoolFactoryCreateFailure.java | 4 ++--
.../java/org/apache/commons/pool2/impl/TestPoolImplUtils.java | 4 ++--
.../org/apache/commons/pool2/impl/TestSoftRefOutOfMemory.java | 2 +-
.../apache/commons/pool2/impl/TestSoftReferenceObjectPool.java | 2 +-
.../pool2/impl/TestSynchronizedPooledObjectFactory.java | 2 +-
.../org/apache/commons/pool2/performance/PerformanceTest.java | 2 +-
.../org/apache/commons/pool2/pool407/KeyedPool407Test.java | 2 +-
.../java/org/apache/commons/pool2/pool407/Pool407Test.java | 2 +-
.../pool2/proxy/AbstractTestProxiedKeyedObjectPool.java | 4 ++--
.../commons/pool2/proxy/AbstractTestProxiedObjectPool.java | 4 ++--
27 files changed, 45 insertions(+), 45 deletions(-)
diff --git a/src/main/java/org/apache/commons/pool2/PoolUtils.java
b/src/main/java/org/apache/commons/pool2/PoolUtils.java
index 65fb8f1e..5345d7dd 100644
--- a/src/main/java/org/apache/commons/pool2/PoolUtils.java
+++ b/src/main/java/org/apache/commons/pool2/PoolUtils.java
@@ -328,7 +328,7 @@ public final class PoolUtils {
* @param <T> type of objects in the pool
*
*/
- private static class ErodingObjectPool<T> implements ObjectPool<T> {
+ private static final class ErodingObjectPool<T> implements ObjectPool<T> {
/** Underlying object pool */
private final ObjectPool<T> pool;
diff --git
a/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java
b/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java
index dcc1bc5c..2f95ef5d 100644
--- a/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java
+++ b/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java
@@ -249,7 +249,7 @@ public abstract class BaseGenericObjectPool<T> extends
BaseObject implements Aut
* Maintains a cache of values for a single metric and reports
* statistics on the cached values.
*/
- private static class StatsStore {
+ private static final class StatsStore {
private static final int NONE = -1;
private final AtomicLong[] values;
diff --git a/src/main/java/org/apache/commons/pool2/impl/EvictionTimer.java
b/src/main/java/org/apache/commons/pool2/impl/EvictionTimer.java
index 0cf43e9b..d57ae2a9 100644
--- a/src/main/java/org/apache/commons/pool2/impl/EvictionTimer.java
+++ b/src/main/java/org/apache/commons/pool2/impl/EvictionTimer.java
@@ -51,7 +51,7 @@ class EvictionTimer {
/**
* Thread factory that creates a daemon thread, with the context class
loader from this class.
*/
- private static class EvictorThreadFactory implements ThreadFactory {
+ private static final class EvictorThreadFactory implements ThreadFactory {
@Override
public Thread newThread(final Runnable runnable) {
@@ -70,7 +70,7 @@ class EvictionTimer {
* Task that removes references to abandoned tasks and shuts
* down the executor if there are no live tasks left.
*/
- private static class Reaper implements Runnable {
+ private static final class Reaper implements Runnable {
@Override
public void run() {
synchronized (EvictionTimer.class) {
@@ -95,7 +95,7 @@ class EvictionTimer {
* no longer reachable, run is no-op.
* @param <R> The kind of Runnable.
*/
- private static class WeakRunner<R extends Runnable> implements Runnable {
+ private static final class WeakRunner<R extends Runnable> implements
Runnable {
private final WeakReference<R> ref;
diff --git
a/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java
b/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java
index 18f82566..daae216f 100644
--- a/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java
+++ b/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java
@@ -97,7 +97,7 @@ public class GenericKeyedObjectPool<K, T> extends
BaseGenericObjectPool<T>
*
* @param <S> type of objects in the pool
*/
- private static class ObjectDeque<S> {
+ private static final class ObjectDeque<S> {
private final LinkedBlockingDeque<PooledObject<S>> idleObjects;
diff --git
a/src/main/java/org/apache/commons/pool2/impl/SecurityManagerCallStack.java
b/src/main/java/org/apache/commons/pool2/impl/SecurityManagerCallStack.java
index 1276e1d7..a695c618 100644
--- a/src/main/java/org/apache/commons/pool2/impl/SecurityManagerCallStack.java
+++ b/src/main/java/org/apache/commons/pool2/impl/SecurityManagerCallStack.java
@@ -40,7 +40,7 @@ public class SecurityManagerCallStack implements CallStack {
/**
* A custom security manager.
*/
- private static class PrivateSecurityManager extends SecurityManager {
+ private static final class PrivateSecurityManager extends SecurityManager {
/**
* Gets the class stack.
@@ -56,7 +56,7 @@ public class SecurityManagerCallStack implements CallStack {
/**
* A snapshot of a class stack.
*/
- private static class Snapshot {
+ private static final class Snapshot {
private final long timestampMillis = System.currentTimeMillis();
private final List<WeakReference<Class<?>>> stack;
diff --git
a/src/main/java/org/apache/commons/pool2/impl/ThrowableCallStack.java
b/src/main/java/org/apache/commons/pool2/impl/ThrowableCallStack.java
index b220ab37..89417b2e 100644
--- a/src/main/java/org/apache/commons/pool2/impl/ThrowableCallStack.java
+++ b/src/main/java/org/apache/commons/pool2/impl/ThrowableCallStack.java
@@ -33,7 +33,7 @@ public class ThrowableCallStack implements CallStack {
/**
* A snapshot of a throwable.
*/
- private static class Snapshot extends Throwable {
+ private static final class Snapshot extends Throwable {
private static final long serialVersionUID = 1L;
private final long timestampMillis = System.currentTimeMillis();
}
diff --git
a/src/test/java/org/apache/commons/pool2/AbstractTestKeyedObjectPool.java
b/src/test/java/org/apache/commons/pool2/AbstractTestKeyedObjectPool.java
index 0a4c3507..0043440b 100644
--- a/src/test/java/org/apache/commons/pool2/AbstractTestKeyedObjectPool.java
+++ b/src/test/java/org/apache/commons/pool2/AbstractTestKeyedObjectPool.java
@@ -160,7 +160,7 @@ public abstract class AbstractTestKeyedObjectPool {
}
}
- private static class TestFactory extends
BaseKeyedPooledObjectFactory<Object, Object> {
+ private static final class TestFactory extends
BaseKeyedPooledObjectFactory<Object, Object> {
@Override
public Object create(final Object key) {
return new Object();
diff --git a/src/test/java/org/apache/commons/pool2/ObjectPoolIssue326.java
b/src/test/java/org/apache/commons/pool2/ObjectPoolIssue326.java
index cf864617..80f9fb3b 100644
--- a/src/test/java/org/apache/commons/pool2/ObjectPoolIssue326.java
+++ b/src/test/java/org/apache/commons/pool2/ObjectPoolIssue326.java
@@ -41,7 +41,7 @@ import
org.apache.commons.pool2.impl.GenericKeyedObjectPoolConfig;
* negatively since you need to run it for a while.
*/
public final class ObjectPoolIssue326 {
- private static class ObjectFactory extends
BaseKeyedPooledObjectFactory<Integer, Object> {
+ private static final class ObjectFactory extends
BaseKeyedPooledObjectFactory<Integer, Object> {
@Override
public Object create(final Integer s) {
return new TestObject();
@@ -53,7 +53,7 @@ public final class ObjectPoolIssue326 {
}
}
- private static class Task<E extends Exception> implements Callable<Object>
{
+ private static final class Task<E extends Exception> implements
Callable<Object> {
private final GenericKeyedObjectPool<Integer, Object> m_pool;
private final int m_key;
@@ -90,7 +90,7 @@ public final class ObjectPoolIssue326 {
}
}
- private static class TestObject {
+ private static final class TestObject {
}
public static void main(final String[] args) {
diff --git a/src/test/java/org/apache/commons/pool2/PoolTest.java
b/src/test/java/org/apache/commons/pool2/PoolTest.java
index d765e0af..cfa0fe09 100644
--- a/src/test/java/org/apache/commons/pool2/PoolTest.java
+++ b/src/test/java/org/apache/commons/pool2/PoolTest.java
@@ -31,10 +31,10 @@ import org.junit.jupiter.api.Test;
public class PoolTest {
- private static class Foo {
+ private static final class Foo {
}
- private static class PooledFooFactory implements PooledObjectFactory<Foo> {
+ private static final class PooledFooFactory implements
PooledObjectFactory<Foo> {
private static final long VALIDATION_WAIT_IN_MILLIS = 1000;
@Override
diff --git
a/src/test/java/org/apache/commons/pool2/TestBaseKeyedPooledObjectFactory.java
b/src/test/java/org/apache/commons/pool2/TestBaseKeyedPooledObjectFactory.java
index 7d8ca9bd..6e2ab825 100644
---
a/src/test/java/org/apache/commons/pool2/TestBaseKeyedPooledObjectFactory.java
+++
b/src/test/java/org/apache/commons/pool2/TestBaseKeyedPooledObjectFactory.java
@@ -25,7 +25,7 @@ import org.junit.jupiter.api.Test;
*/
public class TestBaseKeyedPooledObjectFactory {
- private static class TestFactory extends
BaseKeyedPooledObjectFactory<Object, Object> {
+ private static final class TestFactory extends
BaseKeyedPooledObjectFactory<Object, Object> {
@Override
public Object create(final Object key) {
return null;
diff --git a/src/test/java/org/apache/commons/pool2/TestBaseObjectPool.java
b/src/test/java/org/apache/commons/pool2/TestBaseObjectPool.java
index c4414b2c..bae3e856 100644
--- a/src/test/java/org/apache/commons/pool2/TestBaseObjectPool.java
+++ b/src/test/java/org/apache/commons/pool2/TestBaseObjectPool.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
*/
public class TestBaseObjectPool extends AbstractTestObjectPool {
- private static class TestObjectPool extends BaseObjectPool<Object> {
+ private static final class TestObjectPool extends BaseObjectPool<Object> {
@Override
public Object borrowObject() {
diff --git
a/src/test/java/org/apache/commons/pool2/TestBasePoolableObjectFactory.java
b/src/test/java/org/apache/commons/pool2/TestBasePoolableObjectFactory.java
index 7cb02589..0b6ad1c5 100644
--- a/src/test/java/org/apache/commons/pool2/TestBasePoolableObjectFactory.java
+++ b/src/test/java/org/apache/commons/pool2/TestBasePoolableObjectFactory.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
*/
public class TestBasePoolableObjectFactory {
- private static class TestFactory extends
BasePooledObjectFactory<AtomicInteger> {
+ private static final class TestFactory extends
BasePooledObjectFactory<AtomicInteger> {
@Override
public AtomicInteger create() {
return new AtomicInteger(0);
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestAbandonedKeyedObjectPool.java
b/src/test/java/org/apache/commons/pool2/impl/TestAbandonedKeyedObjectPool.java
index 775b789b..0902347c 100644
---
a/src/test/java/org/apache/commons/pool2/impl/TestAbandonedKeyedObjectPool.java
+++
b/src/test/java/org/apache/commons/pool2/impl/TestAbandonedKeyedObjectPool.java
@@ -78,7 +78,7 @@ public class TestAbandonedKeyedObjectPool {
}
}
- private static class SimpleFactory implements
KeyedPooledObjectFactory<Integer, PooledTestObject> {
+ private static final class SimpleFactory implements
KeyedPooledObjectFactory<Integer, PooledTestObject> {
private final long destroyLatencyMillis;
private final long validateLatencyMillis;
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestAbandonedObjectPool.java
b/src/test/java/org/apache/commons/pool2/impl/TestAbandonedObjectPool.java
index 2e166a8f..a7d008fc 100644
--- a/src/test/java/org/apache/commons/pool2/impl/TestAbandonedObjectPool.java
+++ b/src/test/java/org/apache/commons/pool2/impl/TestAbandonedObjectPool.java
@@ -158,7 +158,7 @@ public class TestAbandonedObjectPool {
}
}
- private static class SimpleFactory implements
PooledObjectFactory<PooledTestObject> {
+ private static final class SimpleFactory implements
PooledObjectFactory<PooledTestObject> {
private final long destroyLatency;
private final long validateLatency;
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestGenericKeyedObjectPool.java
b/src/test/java/org/apache/commons/pool2/impl/TestGenericKeyedObjectPool.java
index 1dfec487..04174017 100644
---
a/src/test/java/org/apache/commons/pool2/impl/TestGenericKeyedObjectPool.java
+++
b/src/test/java/org/apache/commons/pool2/impl/TestGenericKeyedObjectPool.java
@@ -75,7 +75,7 @@ import org.junit.jupiter.params.provider.EnumSource;
*/
public class TestGenericKeyedObjectPool extends AbstractTestKeyedObjectPool {
- private static class DaemonThreadFactory implements ThreadFactory {
+ private static final class DaemonThreadFactory implements ThreadFactory {
@Override
public Thread newThread(final Runnable r) {
final Thread t = new Thread(r);
@@ -84,7 +84,7 @@ public class TestGenericKeyedObjectPool extends
AbstractTestKeyedObjectPool {
}
}
- private static class DummyFactory extends
BaseKeyedPooledObjectFactory<Object, Object> {
+ private static final class DummyFactory extends
BaseKeyedPooledObjectFactory<Object, Object> {
@Override
public Object create(final Object key) {
return null;
@@ -146,7 +146,7 @@ public class TestGenericKeyedObjectPool extends
AbstractTestKeyedObjectPool {
}
}
- private static class ObjectFactory
+ private static final class ObjectFactory
extends BaseKeyedPooledObjectFactory<Integer, Object> {
@Override
@@ -293,7 +293,7 @@ public class TestGenericKeyedObjectPool extends
AbstractTestKeyedObjectPool {
}
}
- private static class SimplePerKeyFactory extends
BaseKeyedPooledObjectFactory<Object, Object> {
+ private static final class SimplePerKeyFactory extends
BaseKeyedPooledObjectFactory<Object, Object> {
final ConcurrentHashMap<Object, AtomicInteger> map = new
ConcurrentHashMap<>();
@Override
@@ -334,7 +334,7 @@ public class TestGenericKeyedObjectPool extends
AbstractTestKeyedObjectPool {
/**
* DefaultEvictionPolicy modified to add latency
*/
- private static class SlowEvictionPolicy<T> extends
DefaultEvictionPolicy<T> {
+ private static final class SlowEvictionPolicy<T> extends
DefaultEvictionPolicy<T> {
private final long delay;
/**
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPool.java
b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPool.java
index 635c9a74..4a09fc77 100644
--- a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPool.java
+++ b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPool.java
@@ -94,7 +94,7 @@ public class TestGenericObjectPool extends TestBaseObjectPool
{
}
}
- private static class CreateErrorFactory extends
BasePooledObjectFactory<String> {
+ private static final class CreateErrorFactory extends
BasePooledObjectFactory<String> {
private final Semaphore semaphore = new Semaphore(0);
@@ -118,7 +118,7 @@ public class TestGenericObjectPool extends
TestBaseObjectPool {
}
}
- private static class CreateFailFactory extends
BasePooledObjectFactory<String> {
+ private static final class CreateFailFactory extends
BasePooledObjectFactory<String> {
private final Semaphore semaphore = new Semaphore(0);
@@ -154,7 +154,7 @@ public class TestGenericObjectPool extends
TestBaseObjectPool {
}
}
- private static class EvictionThread<T> extends Thread {
+ private static final class EvictionThread<T> extends Thread {
private final GenericObjectPool<T> pool;
@@ -217,7 +217,7 @@ public class TestGenericObjectPool extends
TestBaseObjectPool {
}
}
- private static class InvalidFactory
+ private static final class InvalidFactory
extends BasePooledObjectFactory<Object> {
@Override
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolClassLoaders.java
b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolClassLoaders.java
index 851b741b..a237866e 100644
---
a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolClassLoaders.java
+++
b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolClassLoaders.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
public class TestGenericObjectPoolClassLoaders {
- private static class CustomClassLoader extends URLClassLoader {
+ private static final class CustomClassLoader extends URLClassLoader {
private final int n;
CustomClassLoader(final int n) {
@@ -46,7 +46,7 @@ public class TestGenericObjectPoolClassLoaders {
}
}
- private static class CustomClassLoaderObjectFactory extends
+ private static final class CustomClassLoaderObjectFactory extends
BasePooledObjectFactory<URL> {
private final int n;
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolFactoryCreateFailure.java
b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolFactoryCreateFailure.java
index 6d965414..4a7d34ab 100644
---
a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolFactoryCreateFailure.java
+++
b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolFactoryCreateFailure.java
@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Timeout;
*/
public class TestGenericObjectPoolFactoryCreateFailure {
- private static class SingleObjectFactory extends
BasePooledObjectFactory<Object> {
+ private static final class SingleObjectFactory extends
BasePooledObjectFactory<Object> {
private final AtomicBoolean created = new AtomicBoolean();
@Override
@@ -57,7 +57,7 @@ public class TestGenericObjectPoolFactoryCreateFailure {
}
}
- private static class WinnerRunnable implements Runnable {
+ private static final class WinnerRunnable implements Runnable {
private final CountDownLatch barrier;
private final AtomicBoolean failed;
private final GenericObjectPool<Object> pool;
diff --git a/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java
b/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java
index e8aacda9..fc6d5956 100644
--- a/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java
+++ b/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java
@@ -51,7 +51,7 @@ public class TestPoolImplUtils {
// empty by design
}
- private static class NotSimpleFactory extends FactoryF<Integer> {
+ private static final class NotSimpleFactory extends FactoryF<Integer> {
@Override
public Long create() throws Exception {
return null;
@@ -63,7 +63,7 @@ public class TestPoolImplUtils {
}
}
- private static class SimpleFactory extends BasePooledObjectFactory<String>
{
+ private static final class SimpleFactory extends
BasePooledObjectFactory<String> {
@Override
public String create() {
return null;
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestSoftRefOutOfMemory.java
b/src/test/java/org/apache/commons/pool2/impl/TestSoftRefOutOfMemory.java
index 3df3c969..cdb15516 100644
--- a/src/test/java/org/apache/commons/pool2/impl/TestSoftRefOutOfMemory.java
+++ b/src/test/java/org/apache/commons/pool2/impl/TestSoftRefOutOfMemory.java
@@ -56,7 +56,7 @@ public class TestSoftRefOutOfMemory {
}
}
- private static class OomeFactory extends BasePooledObjectFactory<String> {
+ private static final class OomeFactory extends
BasePooledObjectFactory<String> {
private final OomeTrigger trigger;
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestSoftReferenceObjectPool.java
b/src/test/java/org/apache/commons/pool2/impl/TestSoftReferenceObjectPool.java
index a1ff9153..6f4c12f5 100644
---
a/src/test/java/org/apache/commons/pool2/impl/TestSoftReferenceObjectPool.java
+++
b/src/test/java/org/apache/commons/pool2/impl/TestSoftReferenceObjectPool.java
@@ -26,7 +26,7 @@ import org.apache.commons.pool2.TestBaseObjectPool;
*/
public class TestSoftReferenceObjectPool extends TestBaseObjectPool {
- private static class SimpleFactory extends BasePooledObjectFactory<String>
{
+ private static final class SimpleFactory extends
BasePooledObjectFactory<String> {
int counter;
diff --git
a/src/test/java/org/apache/commons/pool2/impl/TestSynchronizedPooledObjectFactory.java
b/src/test/java/org/apache/commons/pool2/impl/TestSynchronizedPooledObjectFactory.java
index 53d7fe3c..30b51ab9 100644
---
a/src/test/java/org/apache/commons/pool2/impl/TestSynchronizedPooledObjectFactory.java
+++
b/src/test/java/org/apache/commons/pool2/impl/TestSynchronizedPooledObjectFactory.java
@@ -24,7 +24,7 @@ import org.apache.commons.pool2.PooledObject;
import org.apache.commons.pool2.PooledObjectFactory;
/**
- * Copies PoolUtil's private static class SynchronizedPooledObjectFactory.
+ * Copies PoolUtil's private static final class
SynchronizedPooledObjectFactory.
*
* A fully synchronized PooledObjectFactory that wraps a PooledObjectFactory
and
* synchronizes access to the wrapped factory methods.
diff --git
a/src/test/java/org/apache/commons/pool2/performance/PerformanceTest.java
b/src/test/java/org/apache/commons/pool2/performance/PerformanceTest.java
index f99e9285..18e2376c 100644
--- a/src/test/java/org/apache/commons/pool2/performance/PerformanceTest.java
+++ b/src/test/java/org/apache/commons/pool2/performance/PerformanceTest.java
@@ -91,7 +91,7 @@ public class PerformanceTest {
}
}
}
- private static class TaskStats {
+ private static final class TaskStats {
public int waiting;
public int complete;
public long totalBorrowNanos;
diff --git
a/src/test/java/org/apache/commons/pool2/pool407/KeyedPool407Test.java
b/src/test/java/org/apache/commons/pool2/pool407/KeyedPool407Test.java
index 9dc4d641..5ac3c4dd 100644
--- a/src/test/java/org/apache/commons/pool2/pool407/KeyedPool407Test.java
+++ b/src/test/java/org/apache/commons/pool2/pool407/KeyedPool407Test.java
@@ -34,7 +34,7 @@ public class KeyedPool407Test extends AbstractPool407Test {
/**
* Borrows from a pool and then immediately returns to that a pool.
*/
- private static class KeyedPool407RoundtripRunnable implements Runnable {
+ private static final class KeyedPool407RoundtripRunnable implements
Runnable {
private final KeyedPool407 pool;
public KeyedPool407RoundtripRunnable(final KeyedPool407 pool) {
diff --git a/src/test/java/org/apache/commons/pool2/pool407/Pool407Test.java
b/src/test/java/org/apache/commons/pool2/pool407/Pool407Test.java
index af18b158..50e833f3 100644
--- a/src/test/java/org/apache/commons/pool2/pool407/Pool407Test.java
+++ b/src/test/java/org/apache/commons/pool2/pool407/Pool407Test.java
@@ -34,7 +34,7 @@ public class Pool407Test extends AbstractPool407Test {
/**
* Borrows from a pool and then immediately returns to that a pool.
*/
- private static class Pool407RoundtripRunnable implements Runnable {
+ private static final class Pool407RoundtripRunnable implements Runnable {
private final Pool407 pool;
public Pool407RoundtripRunnable(final Pool407 pool) {
diff --git
a/src/test/java/org/apache/commons/pool2/proxy/AbstractTestProxiedKeyedObjectPool.java
b/src/test/java/org/apache/commons/pool2/proxy/AbstractTestProxiedKeyedObjectPool.java
index efae51e9..75bfdf2e 100644
---
a/src/test/java/org/apache/commons/pool2/proxy/AbstractTestProxiedKeyedObjectPool.java
+++
b/src/test/java/org/apache/commons/pool2/proxy/AbstractTestProxiedKeyedObjectPool.java
@@ -40,7 +40,7 @@ import org.junit.jupiter.api.Test;
public abstract class AbstractTestProxiedKeyedObjectPool {
- private static class TestKeyedObjectFactory extends
BaseKeyedPooledObjectFactory<String, TestObject> {
+ private static final class TestKeyedObjectFactory extends
BaseKeyedPooledObjectFactory<String, TestObject> {
@Override
public TestObject create(final String key) {
@@ -58,7 +58,7 @@ public abstract class AbstractTestProxiedKeyedObjectPool {
void setData(String data);
}
- private static class TestObjectImpl implements TestObject {
+ private static final class TestObjectImpl implements TestObject {
private String data;
diff --git
a/src/test/java/org/apache/commons/pool2/proxy/AbstractTestProxiedObjectPool.java
b/src/test/java/org/apache/commons/pool2/proxy/AbstractTestProxiedObjectPool.java
index e7972952..f65e53ba 100644
---
a/src/test/java/org/apache/commons/pool2/proxy/AbstractTestProxiedObjectPool.java
+++
b/src/test/java/org/apache/commons/pool2/proxy/AbstractTestProxiedObjectPool.java
@@ -43,7 +43,7 @@ public abstract class AbstractTestProxiedObjectPool {
String getData();
void setData(String data);
}
- private static class TestObjectFactory extends
+ private static final class TestObjectFactory extends
BasePooledObjectFactory<TestObject> {
@Override
@@ -56,7 +56,7 @@ public abstract class AbstractTestProxiedObjectPool {
}
}
- private static class TestObjectImpl implements TestObject {
+ private static final class TestObjectImpl implements TestObject {
private String data;
