This is an automated email from the ASF dual-hosted git repository. garydgregory pushed a commit to branch 1.X in repository https://gitbox.apache.org/repos/asf/commons-beanutils.git
commit d7c4b8b21948f2d634c87ae20e9e4fb4dc39ffeb Author: Gary Gregory <[email protected]> AuthorDate: Sun Jul 26 09:36:32 2026 -0400 Use String.isEmpty() --- .../commons/beanutils/expression/DefaultResolver.java | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/main/java/org/apache/commons/beanutils/expression/DefaultResolver.java b/src/main/java/org/apache/commons/beanutils/expression/DefaultResolver.java index 7105b452..7b2e1751 100644 --- a/src/main/java/org/apache/commons/beanutils/expression/DefaultResolver.java +++ b/src/main/java/org/apache/commons/beanutils/expression/DefaultResolver.java @@ -77,7 +77,7 @@ public class DefaultResolver implements Resolver { */ @Override public int getIndex(final String expression) { - if (expression == null || expression.length() == 0) { + if (expression == null || expression.isEmpty()) { return -1; } for (int i = 0; i < expression.length(); i++) { @@ -91,7 +91,7 @@ public class DefaultResolver implements Resolver { throw new IllegalArgumentException("Missing End Delimiter"); } final String value = expression.substring(i + 1, end); - if (value.length() == 0) { + if (value.isEmpty()) { throw new IllegalArgumentException("No Index Value"); } int index = 0; @@ -116,7 +116,7 @@ public class DefaultResolver implements Resolver { */ @Override public String getKey(final String expression) { - if (expression == null || expression.length() == 0) { + if (expression == null || expression.isEmpty()) { return null; } for (int i = 0; i < expression.length(); i++) { @@ -143,7 +143,7 @@ public class DefaultResolver implements Resolver { */ @Override public String getProperty(final String expression) { - if (expression == null || expression.length() == 0) { + if (expression == null || expression.isEmpty()) { return expression; } for (int i = 0; i < expression.length(); i++) { @@ -164,7 +164,7 @@ public class DefaultResolver implements Resolver { */ @Override public boolean hasNested(final String expression) { - if (expression == null || expression.length() == 0) { + if (expression == null || expression.isEmpty()) { return false; } return remove(expression) != null; @@ -179,7 +179,7 @@ public class DefaultResolver implements Resolver { */ @Override public boolean isIndexed(final String expression) { - if (expression == null || expression.length() == 0) { + if (expression == null || expression.isEmpty()) { return false; } for (int i = 0; i < expression.length(); i++) { @@ -203,7 +203,7 @@ public class DefaultResolver implements Resolver { */ @Override public boolean isMapped(final String expression) { - if (expression == null || expression.length() == 0) { + if (expression == null || expression.isEmpty()) { return false; } for (int i = 0; i < expression.length(); i++) { @@ -227,7 +227,7 @@ public class DefaultResolver implements Resolver { */ @Override public String next(final String expression) { - if (expression == null || expression.length() == 0) { + if (expression == null || expression.isEmpty()) { return null; } boolean indexed = false; @@ -270,7 +270,7 @@ public class DefaultResolver implements Resolver { */ @Override public String remove(final String expression) { - if (expression == null || expression.length() == 0) { + if (expression == null || expression.isEmpty()) { return null; } final String property = next(expression);
