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

Reply via email to