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 ec5fd5f411e3356201fd1e88d8b8d6b5ea3e0964
Author: Gary Gregory <[email protected]>
AuthorDate: Sat May 30 09:07:37 2020 -0400

    Refactor constants.
---
 .../apache/commons/beanutils2/BasicDynaBean.java   | 28 ++++++++++++++++------
 1 file changed, 21 insertions(+), 7 deletions(-)

diff --git a/src/main/java/org/apache/commons/beanutils2/BasicDynaBean.java 
b/src/main/java/org/apache/commons/beanutils2/BasicDynaBean.java
index 20c3145..bb8c2fe 100644
--- a/src/main/java/org/apache/commons/beanutils2/BasicDynaBean.java
+++ b/src/main/java/org/apache/commons/beanutils2/BasicDynaBean.java
@@ -38,6 +38,20 @@ import java.util.Map;
 
 public class BasicDynaBean implements DynaBean, Serializable {
 
+    private static final Short SHORT_ZERO = Short.valueOf((short) 0);
+
+    private static final Long LONG_ZERO = Long.valueOf(0);
+
+    private static final Integer INTEGER_ZERO = Integer.valueOf(0);
+
+    private static final Float FLOAT_ZERO = Float.valueOf((float) 0.0);
+
+    private static final Double DOUBLE_ZERO = Double.valueOf(0.0);
+
+    private static final Character CHARACTER_ZERO = Character.valueOf((char) 
0);
+
+    private static final Byte BYTE_ZERO = Byte.valueOf((byte) 0);
+
     private static final long serialVersionUID = 1L;
 
     /**
@@ -144,19 +158,19 @@ public class BasicDynaBean implements DynaBean, 
Serializable {
         if (type == Boolean.TYPE) {
             return Boolean.FALSE;
         } else if (type == Byte.TYPE) {
-            return Byte.valueOf((byte) 0);
+            return BYTE_ZERO;
         } else if (type == Character.TYPE) {
-            return Character.valueOf((char) 0);
+            return CHARACTER_ZERO;
         } else if (type == Double.TYPE) {
-            return Double.valueOf(0.0);
+            return DOUBLE_ZERO;
         } else if (type == Float.TYPE) {
-            return Float.valueOf((float) 0.0);
+            return FLOAT_ZERO;
         } else if (type == Integer.TYPE) {
-            return Integer.valueOf(0);
+            return INTEGER_ZERO;
         } else if (type == Long.TYPE) {
-            return Long.valueOf(0);
+            return LONG_ZERO;
         } else if (type == Short.TYPE) {
-            return Short.valueOf((short) 0);
+            return SHORT_ZERO;
         } else {
             return null;
         }

Reply via email to