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


The following commit(s) were added to refs/heads/master by this push:
     new 272c917  null check not needed.
272c917 is described below

commit 272c917af1dc6827c3f585e4a6de25a519c067c4
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sun Feb 28 10:07:14 2021 -0500

    null check not needed.
---
 src/main/java/org/apache/commons/ognl/ASTAdd.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/commons/ognl/ASTAdd.java 
b/src/main/java/org/apache/commons/ognl/ASTAdd.java
index b7aaae8..5fde25d 100644
--- a/src/main/java/org/apache/commons/ognl/ASTAdd.java
+++ b/src/main/java/org/apache/commons/ognl/ASTAdd.java
@@ -188,7 +188,7 @@ class ASTAdd
 
                     String expr = children[i].toGetSourceString( context, 
target );
 
-                    if ( ( expr != null && "null".equals( expr ) )
+                    if ( ( "null".equals( expr ) )
                         || ( !ASTConst.class.isInstance( children[i] )
                         && ( expr == null || expr.trim().isEmpty() ) ) )
                     {

Reply via email to