(maven-enforcer) branch master updated: Simplify control flow (#355)

[elharo]

This is an automated email from the ASF dual-hosted git repository.

elharo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-enforcer.git


The following commit(s) were added to refs/heads/master by this push:
     new fe41240  Simplify control flow (#355)
fe41240 is described below

commit fe41240020ec44912e3dcdfb7a72d5207fc4b4cc
Author: Elliotte Rusty Harold <elh...@users.noreply.github.com>
AuthorDate: Sun Feb 23 13:38:53 2025 +0000

    Simplify control flow (#355)
---
 .../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