You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/08/21 15:16:05 UTC

[commons-lang] branch master updated: Revert "ThreadGroupPredicate extends Predicate"

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 4dd0a6b4a Revert "ThreadGroupPredicate extends Predicate<ThreadGroup>"
4dd0a6b4a is described below

commit 4dd0a6b4a67ee66ec754ca2a64200bc73fbd09bf
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Aug 21 11:15:50 2022 -0400

    Revert "ThreadGroupPredicate extends Predicate<ThreadGroup>"
    
    This reverts commit 6cd7bd5ed962ad19fc6af5260e656f35fa2b3d16.
---
 src/changes/changes.xml                                 | 1 -
 src/main/java/org/apache/commons/lang3/ThreadUtils.java | 6 ++----
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index a774319cf..ae8357c84 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -168,7 +168,6 @@ The <action> type attribute can be add,update,fix,remove.
     <action                   type="add" dev="ggregory" due-to="Gary Gregory">Add Streams.of(Iterable&lt;E&gt;).</action>
     <action                   type="add" dev="ggregory" due-to="Gary Gregory">Add Streams.of(Iterator&lt;E&gt;).</action>
     <action issue="LANG-1689" type="add" dev="ggregory" due-to="Joseph Hendrix, Gary Gregory">Simple support for Optional in ObjectUtils#isEmpty() #933.</action>
-    <action                   type="add" dev="ggregory" due-to="Gary Gregory">ThreadGroupPredicate extends Predicate&lt;ThreadGroup&gt;.</action>
     <!-- UPDATE -->
     <action                   type="update" dev="ggregory" due-to="Dependabot, XenoAmess, Gary Gregory">Bump actions/cache from 2.1.4 to 3.0.7 #742, #752, #764, #833, #867.</action>
     <action                   type="update" dev="ggregory" due-to="Dependabot">Bump actions/checkout from 2 to 3 #819, #825, #859.</action>
diff --git a/src/main/java/org/apache/commons/lang3/ThreadUtils.java b/src/main/java/org/apache/commons/lang3/ThreadUtils.java
index 5d8e790dc..369a6c9a4 100644
--- a/src/main/java/org/apache/commons/lang3/ThreadUtils.java
+++ b/src/main/java/org/apache/commons/lang3/ThreadUtils.java
@@ -21,7 +21,6 @@ import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
-import java.util.function.Predicate;
 
 import org.apache.commons.lang3.time.DurationUtils;
 
@@ -90,16 +89,15 @@ public class ThreadUtils {
     /**
      * A predicate for selecting thread groups.
      */
-    // TODO When breaking BC, replace this with Predicate<ThreadGroup>
+    // When breaking BC, replace this with Predicate<ThreadGroup>
     @FunctionalInterface
-    public interface ThreadGroupPredicate extends Predicate<ThreadGroup> {
+    public interface ThreadGroupPredicate {
 
         /**
          * Evaluates this predicate on the given thread group.
          * @param threadGroup the thread group
          * @return {@code true} if the threadGroup matches the predicate, otherwise {@code false}
          */
-        @Override
         boolean test(ThreadGroup threadGroup);
     }