[commons-io] 01/02: Use removeIf() for a more efficient implementation which also deals with the PMD empty while loop error.

[ggregory]

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-io.git

commit 73d32811ce6b6d6fc84a5d62071058db60b7f7dd
Author: Gary Gregory <gardgreg...@gmail.com>
AuthorDate: Tue Jun 14 18:37:28 2022 -0400

    Use removeIf() for a more efficient implementation which also deals with
    the PMD empty while loop error.
---
 .../java/org/apache/commons/io/monitor/FileAlterationMonitor.java     | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/io/monitor/FileAlterationMonitor.java 
b/src/main/java/org/apache/commons/io/monitor/FileAlterationMonitor.java
index df539aa9..9a4b81b0 100644
--- a/src/main/java/org/apache/commons/io/monitor/FileAlterationMonitor.java
+++ b/src/main/java/org/apache/commons/io/monitor/FileAlterationMonitor.java
@@ -129,9 +129,7 @@ public final class FileAlterationMonitor implements 
Runnable {
      */
     public void removeObserver(final FileAlterationObserver observer) {
         if (observer != null) {
-            while (observers.remove(observer)) {
-                // empty
-            }
+            observers.removeIf(observer::equals);
         }
     }