You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ma...@apache.org on 2011/05/31 22:11:02 UTC
svn commit: r1129895 -
/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java
Author: martinkoci
Date: Tue May 31 20:11:01 2011
New Revision: 1129895
URL: http://svn.apache.org/viewvc?rev=1129895&view=rev
Log:
MYFACES-3130 [PERF] Avoid unnecessary AbstractList$Itr instances
Modified:
myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java
Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java?rev=1129895&r1=1129894&r2=1129895&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java Tue May 31 20:11:01 2011
@@ -1853,8 +1853,9 @@ public class ApplicationImpl extends App
if (listenerForList != null) //listeners were found through inspection or from cache, handle them
{
- for (ListenerFor listenerFor : listenerForList)
+ for (int i = 0, size = listenerForList.size(); i < size; i++)
{
+ ListenerFor listenerFor = listenerForList.get(i);
_handleListenerFor(context, inspected, component, listenerFor);
}
}
@@ -1973,8 +1974,9 @@ public class ApplicationImpl extends App
if (dependencyList != null) //resource dependencies were found through inspection or from cache, handle them
{
- for (ResourceDependency dependency : dependencyList)
+ for (int i = 0, size = dependencyList.size(); i < size; i++)
{
+ ResourceDependency dependency = dependencyList.get(i);
if (!rvc.isResourceDependencyAlreadyProcessed(dependency))
{
_handleResourceDependency(context, component, dependency);