You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ba...@apache.org on 2010/07/06 09:13:47 UTC

svn commit: r960828 - /tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/tag/common/core/ForEachSupport.java

Author: bayard
Date: Tue Jul  6 07:13:47 2010
New Revision: 960828

URL: http://svn.apache.org/viewvc?rev=960828&view=rev
Log:
Optimizing to use Boolean.TRUE/FALSE and not new Boolean

Modified:
    tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/tag/common/core/ForEachSupport.java

Modified: tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/tag/common/core/ForEachSupport.java
URL: http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/tag/common/core/ForEachSupport.java?rev=960828&r1=960827&r2=960828&view=diff
==============================================================================
--- tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/tag/common/core/ForEachSupport.java (original)
+++ tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/tag/common/core/ForEachSupport.java Tue Jul  6 07:13:47 2010
@@ -405,8 +405,9 @@ public abstract class ForEachSupport ext
     // returns an iterator over a boolean[] array, wrapping items in Boolean
     protected ForEachIterator toForEachIterator(boolean[] a) {
         Boolean[] wrapped = new Boolean[a.length];
-        for (int i = 0; i < a.length; i++)
-            wrapped[i] = new Boolean(a[i]);
+        for (int i = 0; i < a.length; i++) {
+            wrapped[i] = a[i] ? Boolean.TRUE : Boolean.FALSE;
+        }
         return new SimpleForEachIterator(Arrays.asList(wrapped).iterator());
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org