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-validator.git


The following commit(s) were added to refs/heads/master by this push:
     new d678ed40 Remove noisy parens
d678ed40 is described below

commit d678ed40ea2483a248402d8db57e9e4a78704716
Author: Gary Gregory <[email protected]>
AuthorDate: Fri Jul 7 19:05:27 2023 -0400

    Remove noisy parens
---
 src/main/java/org/apache/commons/validator/Field.java          | 10 +++++-----
 .../java/org/apache/commons/validator/ValidatorResult.java     |  4 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/main/java/org/apache/commons/validator/Field.java 
b/src/main/java/org/apache/commons/validator/Field.java
index 394ab415..27179cc7 100644
--- a/src/main/java/org/apache/commons/validator/Field.java
+++ b/src/main/java/org/apache/commons/validator/Field.java
@@ -276,7 +276,7 @@ public class Field implements Cloneable, Serializable {
      */
     public String getMsg(final String key) {
         final Msg msg = getMessage(key);
-        return (msg == null) ? null : msg.getKey();
+        return msg == null ? null : msg.getKey();
     }
 
     /**
@@ -427,7 +427,7 @@ public class Field implements Cloneable, Serializable {
      * @since 1.1
      */
     public Arg getArg(final String key, final int position) {
-        if ((position >= this.args.length) || (this.args[position] == null)) {
+        if (position >= this.args.length || this.args[position] == null) {
             return null;
         }
 
@@ -435,11 +435,11 @@ public class Field implements Cloneable, Serializable {
 
         // Didn't find default arg so exit, otherwise we would get into
         // infinite recursion
-        if ((arg == null) && key.equals(DEFAULT_ARG)) {
+        if (arg == null && key.equals(DEFAULT_ARG)) {
             return null;
         }
 
-        return (arg == null) ? this.getArg(position) : arg;
+        return arg == null ? this.getArg(position) : arg;
     }
 
     /**
@@ -540,7 +540,7 @@ public class Field implements Cloneable, Serializable {
      * @return Whether the Field is indexed.
      */
     public boolean isIndexed() {
-        return (indexedListProperty != null && !indexedListProperty.isEmpty());
+        return indexedListProperty != null && !indexedListProperty.isEmpty();
     }
 
     /**
diff --git a/src/main/java/org/apache/commons/validator/ValidatorResult.java 
b/src/main/java/org/apache/commons/validator/ValidatorResult.java
index a700d3ec..f3ca9bda 100644
--- a/src/main/java/org/apache/commons/validator/ValidatorResult.java
+++ b/src/main/java/org/apache/commons/validator/ValidatorResult.java
@@ -87,7 +87,7 @@ public class ValidatorResult implements Serializable {
      */
     public boolean isValid(final String validatorName) {
         final ResultStatus status = hAction.get(validatorName);
-        return (status == null) ? false : status.isValid();
+        return status == null ? false : status.isValid();
     }
 
     /**
@@ -97,7 +97,7 @@ public class ValidatorResult implements Serializable {
      */
     public Object getResult(final String validatorName) {
         final ResultStatus status = hAction.get(validatorName);
-        return (status == null) ? null : status.getResult();
+        return status == null ? null : status.getResult();
     }
 
     /**

Reply via email to