You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ma...@apache.org on 2025/09/03 13:01:40 UTC

(commons-pool) branch master updated: Also remove old, unsafe loop.

This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-pool.git


The following commit(s) were added to refs/heads/master by this push:
     new dedd647c Also remove old, unsafe loop.
dedd647c is described below

commit dedd647cb27fd16e1ceef2c067be1bfa9bc4da56
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Sep 3 14:01:35 2025 +0100

    Also remove old, unsafe loop.
---
 src/main/java/org/apache/commons/pool3/impl/EvictionTimer.java | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/src/main/java/org/apache/commons/pool3/impl/EvictionTimer.java b/src/main/java/org/apache/commons/pool3/impl/EvictionTimer.java
index 0bf5f9bf..d730b91f 100644
--- a/src/main/java/org/apache/commons/pool3/impl/EvictionTimer.java
+++ b/src/main/java/org/apache/commons/pool3/impl/EvictionTimer.java
@@ -87,14 +87,6 @@ final class EvictionTimer {
                         iterator.remove();
                     }
                 }
-
-                for (final Entry<WeakReference<BaseGenericObjectPool<?, ?>.Evictor>, WeakRunner<BaseGenericObjectPool<?, ?>.Evictor>> entry : TASK_MAP
-                        .entrySet()) {
-                    if (entry.getKey().get() == null) {
-                        executor.remove(entry.getValue());
-                        TASK_MAP.remove(entry.getKey());
-                    }
-                }
                 if (TASK_MAP.isEmpty() && executor != null) {
                     executor.shutdown();
                     executor.setCorePoolSize(0);