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/23 22:02:15 UTC
svn commit: r1126701 -
/myfaces/core/trunk/api/src/main/java/javax/faces/component/UINamingContainer.java
Author: martinkoci
Date: Mon May 23 20:02:15 2011
New Revision: 1126701
URL: http://svn.apache.org/viewvc?rev=1126701&view=rev
Log:
MYFACES-3130 [PERF] Avoid unnecessary AbstractList$Itr instances
Modified:
myfaces/core/trunk/api/src/main/java/javax/faces/component/UINamingContainer.java
Modified: myfaces/core/trunk/api/src/main/java/javax/faces/component/UINamingContainer.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/main/java/javax/faces/component/UINamingContainer.java?rev=1126701&r1=1126700&r2=1126701&view=diff
==============================================================================
--- myfaces/core/trunk/api/src/main/java/javax/faces/component/UINamingContainer.java (original)
+++ myfaces/core/trunk/api/src/main/java/javax/faces/component/UINamingContainer.java Mon May 23 20:02:15 2011
@@ -172,11 +172,10 @@ public class UINamingContainer extends U
}
}
}
- if (getChildCount() > 0) {
- for (UIComponent child : getChildren()) {
- if (child.visitTree(context, callback)) {
- return true;
- }
+ for (int i = 0, childCount = getChildCount(); i < childCount; i++) {
+ UIComponent child = getChildren().get(i);
+ if (child.visitTree(context, callback)) {
+ return true;
}
}
}