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
commit e58d820174690bfeb6004f6aab7a94467e2ed54d Author: Gary D. Gregory <[email protected]> AuthorDate: Tue Feb 11 07:13:40 2025 -0500 Add Checkstyle ModifierOrder --- src/conf/checkstyle.xml | 1 + .../org/apache/commons/pool2/impl/TestGenericObjectPoolConfig.java | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/conf/checkstyle.xml b/src/conf/checkstyle.xml index 477ccd3d..545cabbd 100644 --- a/src/conf/checkstyle.xml +++ b/src/conf/checkstyle.xml @@ -73,6 +73,7 @@ <module name="LeftCurly" /> <module name="MethodParamPad"/> <module name="MissingOverride" /> + <module name="ModifierOrder" /> <module name="ParenPad" /> <module name="TypecastParenPad" /> <module name="VisibilityModifier"> diff --git a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolConfig.java b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolConfig.java index 464f2743..4d38fb59 100644 --- a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolConfig.java +++ b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPoolConfig.java @@ -31,7 +31,7 @@ import org.junit.jupiter.api.Test; */ public class TestGenericObjectPoolConfig { - public final static class TestEvictionPolicy implements EvictionPolicy<TestObject> { + public static final class TestEvictionPolicy implements EvictionPolicy<TestObject> { @Override public boolean evict(final EvictionConfig config, final PooledObject<TestObject> underTest, final int idleCount) { @@ -40,7 +40,7 @@ public class TestGenericObjectPoolConfig { } - public final static class TestObject { + public static final class TestObject { // empty }
