Fixed - else bug introduced with new error context stack

pugjs · Sep 8, 2011 · df6845e · df6845e
1 parent c74ad7e
commit df6845e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/lib/compiler.js b/lib/compiler.js
@@ -123,7 +123,7 @@ Compiler.prototype = {
    */
 
   visit: function(node){
-    var debug = this.debug && false !== node.debug;
+    var debug = this.debug;
 
     if (debug) {
       this.buf.push('__.unshift({ lineno: ' + node.line
@@ -133,6 +133,13 @@ Compiler.prototype = {
         + ' });');
     }
 
+    // Massive hack to fix our context
+    // stack for - else[ if] etc
+    if (false === node.debug) {
+      this.buf.pop();
+      this.buf.pop();
+    }
+
     this.visitNode(node);
 
     if (debug) this.buf.push('__.shift();');

diff --git a/lib/nodes/code.js b/lib/nodes/code.js
@@ -25,7 +25,7 @@ var Code = module.exports = function Code(val, buffer, escape) {
   this.val = val;
   this.buffer = buffer;
   this.escape = escape;
-  if (/^ *else/.test(val)) this.debug = false;
+  if (val.match(/^ *else/)) this.debug = false;
 };
 
 /**