You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2021/01/17 15:26:58 UTC

[commons-jexl] branch master updated: Use isEmpty().

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


The following commit(s) were added to refs/heads/master by this push:
     new daf986d  Use isEmpty().
daf986d is described below

commit daf986d1558947ae1128594087e8eff34bc65515
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Jan 17 10:26:54 2021 -0500

    Use isEmpty().
---
 src/main/java/org/apache/commons/jexl3/JexlArithmetic.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/commons/jexl3/JexlArithmetic.java b/src/main/java/org/apache/commons/jexl3/JexlArithmetic.java
index 2ecb270..474f624 100644
--- a/src/main/java/org/apache/commons/jexl3/JexlArithmetic.java
+++ b/src/main/java/org/apache/commons/jexl3/JexlArithmetic.java
@@ -1453,7 +1453,7 @@ public class JexlArithmetic {
             return ((AtomicBoolean) val).get();
         } else if (val instanceof String) {
             final String strval = val.toString();
-            return strval.length() > 0 && !"false".equals(strval);
+            return !strval.isEmpty() && !"false".equals(strval);
         } else {
             // non null value is true
             return true;