added option to set compileDebug to false

Signed-off-by: Tj Holowaychuk <tj@vision-media.ca>
pugjs · Jul 15, 2011 · d955560 · d955560
1 parent 7987c87
commit d955560
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 16 deletions.
diff --git a/lib/compiler.js b/lib/compiler.js
@@ -125,7 +125,9 @@ Compiler.prototype = {
    */
 
   visit: function(node){
-    this.line(node);
+    if (this.options.compileDebug !== false) {
+      this.line(node);
+    }
     return this.visitNode(node);
   },
 

diff --git a/lib/jade.js b/lib/jade.js
@@ -178,12 +178,12 @@ function parse(str, options){
       console.log('\n\x1b[1mCompiled Function\x1b[0m:\n\n%s', js.replace(/^/gm, '  '));
     }
 
-    jadeHelpers= exports.helpers;
+    Jade = exports;
 
     try {
       return ''
-        + 'var attrs = jadeHelpers.attrs; \n'
-        + 'var escape = jadeHelpers.escape; \n'
+        + 'var attrs = Jade.helpers.attrs; \n'
+        + 'var escape = Jade.helpers.escape; \n'
         + 'var buf = [];\n'
         + (options.self
           ? 'var self = locals || {}, __ = __ || locals.__;\n' + js
@@ -201,6 +201,9 @@ function parse(str, options){
 /**
  * Compile a `Function` representation of the given jade `str`.
  *
+ * Options:
+ *   - `.compileDebug`     Enable/disable debug helpers such as rethrow (`true`)
+ *
  * @param {String} str
  * @param {Options} options
  * @return {Function}
@@ -214,18 +217,24 @@ exports.compile = function(str, options){
       ? JSON.stringify(options.filename)
       : 'undefined';
 
-  jadeHelpers = exports.helpers;
-
-  // Reduce closure madness by injecting some locals
-  var fn = [
-      'var __ = { lineno: 1, input: ' + input + ', filename: ' + filename + ' };'
-    , 'var rethrow = jadeHelpers.rethrow;'
-    , 'try {'
-    , parse(String(str), options || {})
-    , '} catch (err) {'
-    , '  rethrow(err, __.input, __.filename, __.lineno);'
-    , '}'
-  ].join('\n');
+  Jade = exports;
+
+  var fn = [];
+
+  if (options.compileDebug !== false) {
+    // Reduce closure madness by injecting some locals
+    fn = [
+        'var __ = { lineno: 1, input: ' + input + ', filename: ' + filename + ' };'
+      , 'var rethrow = Jade.helpers.rethrow;'
+      , 'try {'
+      , parse(String(str), options || {})
+      , '} catch (err) {'
+      , '  rethrow(err, __.input, __.filename, __.lineno);'
+      , '}'
+    ].join('\n');
+  } else {
+    fn = parse(String(str), options || {});
+  }
 
   return new Function('locals', fn);
 };

diff --git a/test/jade.test.js b/test/jade.test.js
@@ -867,6 +867,12 @@ module.exports = {
         assert.equal('<p>bar</p>', fn.call({ foo: 'bar' }));
     },
 
+    'test .compile() no debug': function(assert){
+        var fn = jade.compile('p no debug', {compileDebug: false});
+        assert.equal('<p>no debug</p>', fn());
+    },
+
+
     'test null attrs on tag': function(assert){
         var tag = new jade.nodes.Tag('a'),
             name = 'href',