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
The following commit(s) were added to refs/heads/master by this push: new 62ff8930 No need for an empty block 62ff8930 is described below commit 62ff89308f8efd326b84c79aef0531bebdc5ac2a Author: Gary Gregory <gardgreg...@gmail.com> AuthorDate: Tue Jun 14 18:53:45 2022 -0400 No need for an empty block Use inline comments --- .../org/apache/commons/io/monitor/FileAlterationObserver.java | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/main/java/org/apache/commons/io/monitor/FileAlterationObserver.java b/src/main/java/org/apache/commons/io/monitor/FileAlterationObserver.java index 064e2ebe..fe711b21 100644 --- a/src/main/java/org/apache/commons/io/monitor/FileAlterationObserver.java +++ b/src/main/java/org/apache/commons/io/monitor/FileAlterationObserver.java @@ -232,22 +232,21 @@ public class FileAlterationObserver implements Serializable { */ public void checkAndNotify() { - /* fire onStart() */ + // fire onStart() for (final FileAlterationListener listener : listeners) { listener.onStart(this); } - /* fire directory/file events */ + // fire directory/file events final File rootFile = rootEntry.getFile(); if (rootFile.exists()) { checkAndNotify(rootEntry, rootEntry.getChildren(), listFiles(rootFile)); } else if (rootEntry.isExists()) { checkAndNotify(rootEntry, rootEntry.getChildren(), FileUtils.EMPTY_FILE_ARRAY); - } else { - // Didn't exist and still doesn't } + // Else: Didn't exist and still doesn't - /* fire onStop() */ + // fire onStop() for (final FileAlterationListener listener : listeners) { listener.onStop(this); }