remove identity checks from compressed compiler

Author: tj

lib/compress.js

@@ -11,8 +11,6 @@ module.exports = Compiler;
 
 function Compiler(options) {
   options = options || {};
-  this.compress = options.compress;
-  this.indentation = options.indent;
 }
 
 /**
@@ -22,7 +20,7 @@ function Compiler(options) {
 Compiler.prototype.compile = function(node){
   return node.stylesheet
     .rules.map(this.visit, this)
-    .join(this.compress ? '' : '\n\n');
+    .join('');
 };
 
 /**
@@ -39,7 +37,6 @@ Compiler.prototype.visit = function(node){
 
 Compiler.prototype.comment = function(node){
   if (this.compress) return '';
-  return this.indent() + '/*' + node.comment + '*/';
 };
 
 /**
@@ -55,21 +52,11 @@ Compiler.prototype.import = function(node){
  */
 
 Compiler.prototype.media = function(node){
-  if (this.compress) {
-    return '@media '
-      + node.media
-      + '{'
-      + node.rules.map(this.visit, this).join('')
-      + '}';
-  }
-
   return '@media '
     + node.media
-    + ' {\n'
-    + this.indent(1)
-    + node.rules.map(this.visit, this).join('\n\n')
-    + this.indent(-1)
-    + '\n}';
+    + '{'
+    + node.rules.map(this.visit, this).join('')
+    + '}';
 };
 
 /**
@@ -79,31 +66,18 @@ Compiler.prototype.media = function(node){
 Compiler.prototype.document = function(node){
   var doc = '@' + (node.vendor || '') + 'document ' + node.document;
 
-  if (this.compress) {
-    return doc
-      + '{'
-      + node.rules.map(this.visit, this).join('')
-      + '}';
-  }
-
-  return doc + ' '
-    + ' {\n'
-    + this.indent(1)
-    + node.rules.map(this.visit, this).join('\n\n')
-    + this.indent(-1)
-    + '\n}';
+  return doc
+    + '{'
+    + node.rules.map(this.visit, this).join('')
+    + '}';
 };
 
 /**
  * Visit charset node.
  */
 
 Compiler.prototype.charset = function(node){
-  if (this.compress) {
-    return '@charset ' + node.charset + ';';
-  }
-
-  return '@charset ' + node.charset + ';\n';
+  return '@charset ' + node.charset + ';';
 };
 
 /**
@@ -125,24 +99,12 @@ Compiler.prototype.supports = function(node){
  */
 
 Compiler.prototype.keyframes = function(node){
-  if (this.compress) {
-    return '@'
-      + (node.vendor || '')
-      + 'keyframes '
-      + node.name
-      + '{'
-      + node.keyframes.map(this.visit, this).join('')
-      + '}';
-  }
-
   return '@'
     + (node.vendor || '')
     + 'keyframes '
     + node.name
-    + ' {\n'
-    + this.indent(1)
-    + node.keyframes.map(this.visit, this).join('\n')
-    + this.indent(-1)
+    + '{'
+    + node.keyframes.map(this.visit, this).join('')
     + '}';
 };
 
@@ -153,20 +115,10 @@ Compiler.prototype.keyframes = function(node){
 Compiler.prototype.keyframe = function(node){
   var decls = node.declarations;
 
-  if (this.compress) {
-    return node.values.join(',')
-      + '{'
-      + decls.map(this.visit, this).join('')
-      + '}';
-  }
-
-  return this.indent()
-    + node.values.join(', ')
-    + ' {\n'
-    + this.indent(1)
-    + decls.map(this.visit, this).join('\n')
-    + this.indent(-1)
-    + '\n' + this.indent() + '}\n';
+  return node.values.join(',')
+    + '{'
+    + decls.map(this.visit, this).join('')
+    + '}';
 };
 
 /**
@@ -191,49 +143,20 @@ Compiler.prototype.page = function(node){
  */
 
 Compiler.prototype.rule = function(node){
-  var indent = this.indent();
   var decls = node.declarations;
+  if (!decls.length) return '';
 
-  if (this.compress) {
-    if (!decls.length) return '';
-
-    return node.selectors.join(',')
-      + '{'
-      + decls.map(this.visit, this).join('')
-      + '}';
-  }
-
-  return node.selectors.map(function(s){ return indent + s }).join(',\n')
-    + ' {\n'
-    + this.indent(1)
-    + decls.map(this.visit, this).join('\n')
-    + this.indent(-1)
-    + '\n' + this.indent() + '}';
+  return node.selectors.join(',')
+    + '{'
+    + decls.map(this.visit, this).join('')
+    + '}';
 };
 
 /**
  * Visit declaration node.
  */
 
 Compiler.prototype.declaration = function(node){
-  if (this.compress) {
-    return node.property + ':' + node.value + ';';
-  }
-
-  return this.indent() + node.property + ': ' + node.value + ';';
+  return node.property + ':' + node.value + ';';
 };
 
-/**
- * Increase, decrease or return current indentation.
- */
-
-Compiler.prototype.indent = function(level) {
-  this.level = this.level || 1;
-
-  if (null != level) {
-    this.level += level;
-    return '';
-  }
-
-  return Array(this.level).join(this.indentation || '  ');
-};