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


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

commit 8c4a5322dfd51a585c346984e825c1cd6eb268c3
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sat Jan 16 22:02:23 2021 -0500

    Use isEmpty().
---
 src/main/java/org/apache/commons/beanutils2/LazyDynaList.java         | 4 ++--
 .../org/apache/commons/beanutils2/converters/AbstractConverter.java   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/apache/commons/beanutils2/LazyDynaList.java 
b/src/main/java/org/apache/commons/beanutils2/LazyDynaList.java
index 4906a50..689d17d 100644
--- a/src/main/java/org/apache/commons/beanutils2/LazyDynaList.java
+++ b/src/main/java/org/apache/commons/beanutils2/LazyDynaList.java
@@ -509,7 +509,7 @@ public class LazyDynaList extends ArrayList<Object> {
         }
 
         final boolean changeType = this.elementType != null && 
!this.elementType.equals(elementType);
-        if (changeType && size() > 0) {
+        if (changeType && !isEmpty()) {
             throw new IllegalStateException("Element Type cannot be reset");
         }
 
@@ -561,7 +561,7 @@ public class LazyDynaList extends ArrayList<Object> {
             throw new IllegalArgumentException("Element DynaClass is missing");
         }
 
-        if (size() > 0) {
+        if (!isEmpty()) {
             throw new IllegalStateException("Element DynaClass cannot be 
reset");
         }
 
diff --git 
a/src/main/java/org/apache/commons/beanutils2/converters/AbstractConverter.java 
b/src/main/java/org/apache/commons/beanutils2/converters/AbstractConverter.java
index f331cc0..2f72db0 100644
--- 
a/src/main/java/org/apache/commons/beanutils2/converters/AbstractConverter.java
+++ 
b/src/main/java/org/apache/commons/beanutils2/converters/AbstractConverter.java
@@ -231,7 +231,7 @@ public abstract class AbstractConverter implements 
Converter {
         }
         if (value instanceof Collection) {
             final Collection<?> collection = (Collection<?>)value;
-            if (collection.size() > 0) {
+            if (!collection.isEmpty()) {
                 return collection.iterator().next();
             }
             return null;

Reply via email to