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 4b2743b4266e4c94436c38942b961a73ee4d60b0
Author: Gary Gregory <[email protected]>
AuthorDate: Fri Nov 21 20:51:44 2025 +0000

    Merge if statements
---
 .../org/apache/commons/pool2/impl/TestDefaultPooledObject.java     | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java 
b/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java
index a71c8596..b4e7ee91 100644
--- a/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java
+++ b/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java
@@ -49,11 +49,8 @@ class TestDefaultPooledObject {
         final ExecutorService executor = 
Executors.newFixedThreadPool(Runtime.getRuntime().availableProcessors() * 3);
         final Runnable allocateAndDeallocateTask = () -> {
             for (int i1 = 0; i1 < 10000; i1++) {
-                if (dpo.getIdleDuration().isNegative() || 
dpo.getIdleTime().isNegative()) {
-                    negativeIdleTimeReturned.set(true);
-                    break;
-                }
-                if (dpo.getIdleDuration().isNegative() || 
dpo.getIdleTime().isNegative()) {
+                if (dpo.getIdleDuration().isNegative() || 
dpo.getIdleTime().isNegative() || dpo.getIdleDuration().isNegative()
+                        || dpo.getIdleTime().isNegative()) {
                     negativeIdleTimeReturned.set(true);
                     break;
                 }

Reply via email to