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
commit 135b783c034f487c969afdf068de69e0d5f8c918 Author: Gary Gregory <[email protected]> AuthorDate: Sun Aug 18 08:25:56 2024 -0400 Use final USe else if --- src/main/java/org/apache/commons/beanutils2/LazyDynaList.java | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/main/java/org/apache/commons/beanutils2/LazyDynaList.java b/src/main/java/org/apache/commons/beanutils2/LazyDynaList.java index cc0f8fe4..0de80d4e 100644 --- a/src/main/java/org/apache/commons/beanutils2/LazyDynaList.java +++ b/src/main/java/org/apache/commons/beanutils2/LazyDynaList.java @@ -316,7 +316,7 @@ public class LazyDynaList extends ArrayList<Object> { return false; } - ensureCapacity((Math.max(index, size())) + collection.size()); + ensureCapacity(Math.max(index, size()) + collection.size()); // Call "transform" with first element, before // List is "grown" to ensure the correct DynaClass @@ -433,11 +433,12 @@ public class LazyDynaList extends ArrayList<Object> { if (WrapDynaBean.class.isAssignableFrom(elementDynaBeanType)) { this.elementType = ((WrapDynaBean)dynaBean).getInstance().getClass(); this.wrapDynaClass = (WrapDynaClass)elementDynaClass; - } else if (LazyDynaMap.class.isAssignableFrom(elementDynaBeanType)) { - this.elementType = ((LazyDynaMap)dynaBean).getMap().getClass(); - this.elementDynaClass = elementDynaClass; } else { - this.elementType = dynaBean.getClass(); + if (LazyDynaMap.class.isAssignableFrom(elementDynaBeanType)) { + this.elementType = ((LazyDynaMap)dynaBean).getMap().getClass(); + } else { + this.elementType = dynaBean.getClass(); + } this.elementDynaClass = elementDynaClass; } } catch (final Exception e) {
