Test and fix for stack overflow when rendering very large arrays

pepve · Aug 20, 2012 · fc77a5c · fc77a5c
1 parent abfe811
commit fc77a5c
Show file tree

Hide file tree

Showing 5 changed files with 10,018 additions and 1 deletion.
diff --git a/lib/mu/renderer.js b/lib/mu/renderer.js
@@ -12,7 +12,11 @@ function render(tokens, context, partials, stream, callback) {
   return _render(tokens, context, partials, stream, callback);
 }
 
-function _render(tokens, context, partials, stream, callback) {
+function _render() {
+  process.nextTick(Function.prototype.bind.apply(__render, [this].concat(Array.prototype.slice.call(arguments))));
+}
+
+function __render(tokens, context, partials, stream, callback) {
   if (tokens[0] !== 'multi') {
     throw new Error('Mu - WTF did you give me? I expected mustache tokens.');
   }

diff --git a/test/examples/tenthousand.html b/test/examples/tenthousand.html
@@ -0,0 +1,2 @@
+{{#foo}}{{value}}
+{{/foo}}
diff --git a/test/examples/tenthousand.js b/test/examples/tenthousand.js
@@ -0,0 +1,9 @@
+(function () {
+	var result = { foo: [] };
+
+	for (var i = 0; i < 10000; i++) {
+		result.foo.push({ value: i });
+	}
+
+	return result;
+}())