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


The following commit(s) were added to refs/heads/master by this push:
     new 26aaab7  No need to nest in else.
26aaab7 is described below

commit 26aaab7e1ee89a382b8a33d7f248e5327c703e94
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Mar 5 14:15:03 2021 -0500

    No need to nest in else.
---
 .../main/java/org/apache/commons/digester3/ExtendedBaseRules.java | 2 +-
 .../java/org/apache/commons/digester3/xmlrules/CallParamRule.java | 8 ++------
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git 
a/core/src/main/java/org/apache/commons/digester3/ExtendedBaseRules.java 
b/core/src/main/java/org/apache/commons/digester3/ExtendedBaseRules.java
index 4434f1b..d455ff7 100644
--- a/core/src/main/java/org/apache/commons/digester3/ExtendedBaseRules.java
+++ b/core/src/main/java/org/apache/commons/digester3/ExtendedBaseRules.java
@@ -465,7 +465,7 @@ public class ExtendedBaseRules
                     }
                     return -1;
                 }
-                else if ( i2 == null )
+                if ( i2 == null )
                 {
                     return 1;
                 }
diff --git 
a/core/src/main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java 
b/core/src/main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java
index a8236ba..c03e038 100644
--- 
a/core/src/main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java
+++ 
b/core/src/main/java/org/apache/commons/digester3/xmlrules/CallParamRule.java
@@ -63,15 +63,11 @@ final class CallParamRule
         }
         else
         {
-            if ( fromStack == null )
-            {
-                builder.fromAttribute( attributeName );
-            }
-            else
-            {
+            if ( fromStack != null ) {
                 // specifying both from-stack and attribute name is not allowed
                 throw new RuntimeException( "Attributes from-stack and 
attrname cannot both be present." );
             }
+            builder.fromAttribute( attributeName );
         }
     }
 

Reply via email to