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 cb99f50  Use final.
cb99f50 is described below

commit cb99f50a7df0667abd0c70037ef1b3ca99cb4c8a
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Nov 20 22:02:48 2020 -0500

    Use final.
---
 src/main/java/org/apache/commons/beanutils2/LazyDynaClass.java        | 2 +-
 src/main/java/org/apache/commons/beanutils2/MethodUtils.java          | 2 +-
 src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java    | 4 ++--
 .../org/apache/commons/beanutils2/converters/AbstractConverter.java   | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/commons/beanutils2/LazyDynaClass.java 
b/src/main/java/org/apache/commons/beanutils2/LazyDynaClass.java
index 9c29af5..2205b2e 100644
--- a/src/main/java/org/apache/commons/beanutils2/LazyDynaClass.java
+++ b/src/main/java/org/apache/commons/beanutils2/LazyDynaClass.java
@@ -284,7 +284,7 @@ public class LazyDynaClass extends BasicDynaClass 
implements MutableDynaClass  {
         final DynaProperty[] oldProperties = getDynaProperties();
         final DynaProperty[] newProperties = new 
DynaProperty[oldProperties.length-1];
         int j = 0;
-        for (DynaProperty oldProperty : oldProperties) {
+        for (final DynaProperty oldProperty : oldProperties) {
             if (!name.equals(oldProperty.getName())) {
                 newProperties[j] = oldProperty;
                 j++;
diff --git a/src/main/java/org/apache/commons/beanutils2/MethodUtils.java 
b/src/main/java/org/apache/commons/beanutils2/MethodUtils.java
index 32d86d1..60f5def 100644
--- a/src/main/java/org/apache/commons/beanutils2/MethodUtils.java
+++ b/src/main/java/org/apache/commons/beanutils2/MethodUtils.java
@@ -858,7 +858,7 @@ public class MethodUtils {
 
             // Check the implemented interfaces of the parent class
             final Class<?>[] interfaces = clazz.getInterfaces();
-            for (Class<?> anInterface : interfaces) {
+            for (final Class<?> anInterface : interfaces) {
 
                 // Is this interface public?
                 if (!Modifier.isPublic(anInterface.getModifiers())) {
diff --git a/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java 
b/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java
index fb12188..eaa2a39 100644
--- a/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java
+++ b/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java
@@ -2061,7 +2061,7 @@ public class PropertyUtilsBean {
         } catch (final NullPointerException | IllegalArgumentException cause) {
             // JDK 1.3 and JDK 1.4 throw NullPointerException if an argument is
             // null for a primitive value (JDK 1.5+ throw 
IllegalArgumentException)
-            StringBuilder valueString = new StringBuilder();
+            final StringBuilder valueString = new StringBuilder();
             if (values != null) {
                 for (int i = 0; i < values.length; i++) {
                     if (i>0) {
@@ -2074,7 +2074,7 @@ public class PropertyUtilsBean {
                     }
                 }
             }
-            StringBuilder expectedString = new StringBuilder();
+            final StringBuilder expectedString = new StringBuilder();
             final Class<?>[] parTypes = method.getParameterTypes();
             if (parTypes != null) {
                 for (int i = 0; i < parTypes.length; i++) {
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 dff8337..c2b6770 100644
--- 
a/src/main/java/org/apache/commons/beanutils2/converters/AbstractConverter.java
+++ 
b/src/main/java/org/apache/commons/beanutils2/converters/AbstractConverter.java
@@ -423,7 +423,7 @@ public abstract class AbstractConverter implements 
Converter {
                 elementType = elementType .getComponentType();
                 count++;
             }
-            StringBuilder typeNameBuilder = new 
StringBuilder(elementType.getName());
+            final StringBuilder typeNameBuilder = new 
StringBuilder(elementType.getName());
             for (int i = 0; i < count; i++) {
                 typeNameBuilder.append("[]");
             }

Reply via email to