This is an automated email from the ASF dual-hosted git repository. elharo pushed a commit to branch bug in repository https://gitbox.apache.org/repos/asf/maven-enforcer.git
commit 43c8cabc94a16218813039038449caf89a783d5d Author: Elliotte Rusty Harold <elh...@ibiblio.org> AuthorDate: Sat Feb 22 17:10:07 2025 -0500 Simplify control flow --- .../org/apache/maven/enforcer/rules/files/RequireFilesDontExist.java | 2 +- .../java/org/apache/maven/enforcer/rules/files/RequireFilesExist.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/files/RequireFilesDontExist.java b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/files/RequireFilesDontExist.java index 116dd04..539bd45 100644 --- a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/files/RequireFilesDontExist.java +++ b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/files/RequireFilesDontExist.java @@ -30,7 +30,7 @@ public final class RequireFilesDontExist extends AbstractRequireFiles { @Override boolean checkFile(File file) { // if we get here and the handle is null, treat it as a success - return file == null ? true : !file.exists(); + return file == null || !file.exists(); } @Override diff --git a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/files/RequireFilesExist.java b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/files/RequireFilesExist.java index 265cf25..a34ef90 100644 --- a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/files/RequireFilesExist.java +++ b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/files/RequireFilesExist.java @@ -31,7 +31,7 @@ public final class RequireFilesExist extends AbstractRequireFiles { @Override boolean checkFile(File file) { // if we get here and the handle is null, treat it as a success - return file == null ? true : file.exists() && osIndependentNameMatch(file); + return file == null || (file.exists() && osIndependentNameMatch(file)); } @Override