vndb/mithril-vndb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #338 from AlexGalays/next

Browse source 
Performance tuning: Array.push instead of Array.concat
This commit is contained in:
Leo Horie 2014-11-13 20:31:52 -05:00
commit a4b589ab36
1 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

6
mithril.js
View file

@ -194,7 +194,11 @@ Mithril = m = new function app(window, undefined) {
//update the list of DOM nodes by collecting the nodes from each item
for (var i = 0; i < data.length; i++) {
if (cached[i] != null) nodes = nodes.concat(cached[i].nodes)
if (cached[i] != null) {
for (var j = 0; j < cached[i].nodes.length; j++) {
nodes.push(cached[i].nodes[j]);
}
}
}
//remove items from the end of the array if the new array is shorter than the old one
//if errors ever happen here, the issue is most likely a bug in the construction of the `cached` data structure somewhere earlier in the program