You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by tn...@apache.org on 2013/01/20 22:48:58 UTC

svn commit: r1435980 - /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/MapUtils.java

Author: tn
Date: Sun Jan 20 21:48:58 2013
New Revision: 1435980

URL: http://svn.apache.org/viewvc?rev=1435980&view=rev
Log:
Findbugs fixes: use e.g. Integer.valueOf instead of new Integer and similar.

Modified:
    commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/MapUtils.java

Modified: commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/MapUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/MapUtils.java?rev=1435980&r1=1435979&r2=1435980&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/MapUtils.java (original)
+++ commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/MapUtils.java Sun Jan 20 21:48:58 2013
@@ -218,7 +218,7 @@ public class MapUtils {
         if (answer instanceof Byte) {
             return (Byte) answer;
         }
-        return new Byte(answer.byteValue());
+        return Byte.valueOf(answer.byteValue());
     }
 
     /**
@@ -238,7 +238,7 @@ public class MapUtils {
         if (answer instanceof Short) {
             return (Short) answer;
         }
-        return new Short(answer.shortValue());
+        return Short.valueOf(answer.shortValue());
     }
 
     /**
@@ -258,7 +258,7 @@ public class MapUtils {
         if (answer instanceof Integer) {
             return (Integer) answer;
         }
-        return new Integer(answer.intValue());
+        return Integer.valueOf(answer.intValue());
     }
 
     /**
@@ -278,7 +278,7 @@ public class MapUtils {
         if (answer instanceof Long) {
             return (Long) answer;
         }
-        return new Long(answer.longValue());
+        return Long.valueOf(answer.longValue());
     }
 
     /**
@@ -298,7 +298,7 @@ public class MapUtils {
         if (answer instanceof Float) {
             return (Float) answer;
         }
-        return new Float(answer.floatValue());
+        return Float.valueOf(answer.floatValue());
     }
 
     /**
@@ -318,7 +318,7 @@ public class MapUtils {
         if (answer instanceof Double) {
             return (Double) answer;
         }
-        return new Double(answer.doubleValue());
+        return Double.valueOf(answer.doubleValue());
     }
 
     /**