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); } }