You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ol...@apache.org on 2011/05/18 11:45:28 UTC

svn commit: r1124148 - in /incubator/ognl/trunk/src/main/java/org/apache/commons/ognl: IntHashMap.java ListPropertyAccessor.java MapPropertyAccessor.java SetPropertyAccessor.java

Author: olamy
Date: Wed May 18 09:45:27 2011
New Revision: 1124148

URL: http://svn.apache.org/viewvc?rev=1124148&view=rev
Log:
use Integer.valueOf rather than new Integer to use vm caching

Modified:
    incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/IntHashMap.java
    incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java
    incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java
    incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/SetPropertyAccessor.java

Modified: incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/IntHashMap.java
URL: http://svn.apache.org/viewvc/incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/IntHashMap.java?rev=1124148&r1=1124147&r2=1124148&view=diff
==============================================================================
--- incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/IntHashMap.java (original)
+++ incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/IntHashMap.java Wed May 18 09:45:27 2011
@@ -113,7 +113,7 @@ public class IntHashMap
                 Entry e = entry;
 
                 entry = e.next;
-                return keys ? new Integer( e.key ) : e.value;
+                return keys ? Integer.valueOf( e.key ) : e.value;
             }
             throw new NoSuchElementException( "IntHashMapIterator" );
         }

Modified: incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java
URL: http://svn.apache.org/viewvc/incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java?rev=1124148&r1=1124147&r2=1124148&view=diff
==============================================================================
--- incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java (original)
+++ incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java Wed May 18 09:45:27 2011
@@ -48,7 +48,7 @@ public class ListPropertyAccessor
 
             if ( name.equals( "size" ) )
             {
-                result = new Integer( list.size() );
+                result = Integer.valueOf( list.size() );
             }
             else
             {

Modified: incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java
URL: http://svn.apache.org/viewvc/incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java?rev=1124148&r1=1124147&r2=1124148&view=diff
==============================================================================
--- incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java (original)
+++ incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/MapPropertyAccessor.java Wed May 18 09:45:27 2011
@@ -58,7 +58,7 @@ public class MapPropertyAccessor
         {
             if ( name.equals( "size" ) )
             {
-                result = new Integer( map.size() );
+                result = Integer.valueOf( map.size() );
             }
             else
             {

Modified: incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/SetPropertyAccessor.java
URL: http://svn.apache.org/viewvc/incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/SetPropertyAccessor.java?rev=1124148&r1=1124147&r2=1124148&view=diff
==============================================================================
--- incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/SetPropertyAccessor.java (original)
+++ incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/SetPropertyAccessor.java Wed May 18 09:45:27 2011
@@ -43,7 +43,7 @@ public class SetPropertyAccessor
 
             if ( name.equals( "size" ) )
             {
-                result = new Integer( set.size() );
+                result = Integer.valueOf( set.size() );
             }
             else
             {