You are viewing a plain text version of this content. The canonical link for it is here.
Posted to xap-commits@incubator.apache.org by jm...@apache.org on 2007/10/03 01:22:49 UTC
svn commit: r581466 -
/incubator/xap/trunk/codebase/src/xap/taghandling/AbstractTagImpl.js
Author: jmargaris
Date: Tue Oct 2 18:22:49 2007
New Revision: 581466
URL: http://svn.apache.org/viewvc?rev=581466&view=rev
Log:
efficiency changes
Modified:
incubator/xap/trunk/codebase/src/xap/taghandling/AbstractTagImpl.js
Modified: incubator/xap/trunk/codebase/src/xap/taghandling/AbstractTagImpl.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/codebase/src/xap/taghandling/AbstractTagImpl.js?rev=581466&r1=581465&r2=581466&view=diff
==============================================================================
--- incubator/xap/trunk/codebase/src/xap/taghandling/AbstractTagImpl.js (original)
+++ incubator/xap/trunk/codebase/src/xap/taghandling/AbstractTagImpl.js Tue Oct 2 18:22:49 2007
@@ -282,8 +282,10 @@
var session = this.getSession();
var contentHandler = session.getUiDocumentHandler();
- var children = new Array(e.childNodes.length);
- for ( var i = 0; i<children.length; i++){
+ var length = e.childNodes.length;
+ if (!length) return;
+ var children = new Array(length);
+ for ( var i = 0; i<length; i++){
children[i] = e.childNodes[i];
}
//TODO we need this for iterator but we really need a more efficient