Tabs and code style

.gitignore

@@ -1,3 +1,2 @@
 node_modules
 optipng/
-

bin/optipng-bin

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
-
 var spawn = require('child_process').spawn;
 var binPath = require('../lib/optipng-bin').path;
 
 spawn(binPath, process.argv.slice(2), { stdio: 'inherit' })
-  .on('exit', process.exit);
+	.on('exit', process.exit);

lib/optipng-bin.js

@@ -2,19 +2,19 @@ var path = require('path');
 var execSync = require('exec-sync');
 
 if (process.platform === 'darwin') {
-    var version = execSync('sw_vers -productVersion');
+	var version = execSync('sw_vers -productVersion');
 
-    if (/10.8/.test(version)) {
-        exports.path = path.join(__dirname, '../vendor/osx/10.8', 'optipng');
-    } else if (/10.7/.test(version)) {
-        exports.path = path.join(__dirname, '../vendor/osx/10.7', 'optipng');
-    }
+	if (/10.8/.test(version)) {
+		exports.path = path.join(__dirname, '../vendor/osx/10.8', 'optipng');
+	} else if (/10.7/.test(version)) {
+		exports.path = path.join(__dirname, '../vendor/osx/10.7', 'optipng');
+	}
 } else if (process.platform === 'linux') {
-    if (process.arch === 'x64') {
-        exports.path = path.join(__dirname, '../vendor/linux/x64', 'optipng');
-    } else {
-        exports.path = path.join(__dirname, '../vendor/linux/x86', 'optipng');
-    }
+	if (process.arch === 'x64') {
+		exports.path = path.join(__dirname, '../vendor/linux/x64', 'optipng');
+	} else {
+		exports.path = path.join(__dirname, '../vendor/linux/x86', 'optipng');
+	}
 } else if (process.platform === 'win32') {
 	exports.path = path.join(__dirname, '../vendor/win32', 'optipng.exe');
 } else {

rebuild-optipng.js

@@ -1,29 +1,27 @@
+'use strict';
 var exec = require('child_process').exec;
 var colors = require('colors');
 var binPath = require('./lib/optipng-bin.js').path;
 var which = require('which');
 var path = require('path');
 
 
-which('make', function(err, makepath){
+which('make', function (err) {
     if (err) {
-        console.log(err.red);
-        return;
+        return console.log(err.red);
     }
 
     if (process.platform === 'darwin' || process.platform === 'linux') {
         var binDir = path.dirname(binPath);
         var buildScript = 'make clean &&' +
                           './configure --with-system-zlib --bindir=' + binDir  + ' --mandir=man && ' +
                           'make install';
-        exec(buildScript, {cwd: './optipng/'}, function(err, stdout, stderr) {
+        exec(buildScript, { cwd: './optipng/' }, function (err) {
             if (err) {
-                console.log(err.red);
-                return;
+                return console.log(err.red);
             }
 
             console.log('OptiPNG rebuilt successfully'.green);
         });
     }
 });
-

test/test-optipng-build.js

@@ -1,19 +1,18 @@
-/*global describe, it, after */
+/*global describe, it */
 'use strict';
-
 var assert = require('assert');
 var exec = require('child_process').exec;
 var binPath = require('../lib/optipng-bin.js').path;
 var fs = require('fs');
 
-describe('OptiPNG rebuild', function() {
-	it('it should rebuild the optipng binaries', function(cb) {
-        this.timeout(15000); // give this test time to build
-        var origCTime = fs.statSync(binPath).ctime;
-		exec('node rebuild-optipng.js', {}, function(err){
-            var actualCTime = fs.statSync(binPath).ctime;
-            assert(actualCTime !== origCTime);
-            cb(err);
-        }).path;
+describe('OptiPNG rebuild', function () {
+	it('it should rebuild the optipng binaries', function (cb) {
+		this.timeout(15000); // give this test time to build
+		var origCTime = fs.statSync(binPath).ctime;
+		exec('node rebuild-optipng.js', {}, function (err) {
+			var actualCTime = fs.statSync(binPath).ctime;
+			assert(actualCTime !== origCTime);
+			cb(err);
+		}).path;
 	});
 });

test/test-optipng-path.js

@@ -1,35 +1,34 @@
 /*global describe, it, after */
 'use strict';
-
 var assert = require('assert');
 var path = require('path');
 var fs = require('fs');
 var execFile = require('child_process').execFile;
 
-describe('OptiPNG', function() {
-	after(function() {
+describe('OptiPNG', function () {
+	after(function () {
 		fs.unlinkSync('test/minified.png');
 	});
 
-	it('should return path to OptiPNG binary', function(cb) {
+	it('should return path to OptiPNG binary', function (cb) {
 		var binPath = require('../lib/optipng-bin.js').path;
 
-		execFile(binPath, ['-v'], function(err, stdout) {
+		execFile(binPath, ['-v'], function (err, stdout) {
 			assert(stdout.indexOf('OptiPNG') !== -1);
 			cb();
 		});
 	});
 
-	it('should proxy OptiPNG', function(cb) {
+	it('should proxy OptiPNG', function (cb) {
 		var binPath = path.join(__dirname, '../bin/optipng-bin');
 
-		execFile(binPath, ['-v'], function(err, stdout) {
+		execFile(binPath, ['-v'], function (err, stdout) {
 			assert(stdout.indexOf('OptiPNG') !== -1);
 			cb();
 		});
 	});
 
-	it('should minify a .png', function(cb) {
+	it('should minify a .png', function (cb) {
 		var binPath = path.join(__dirname, '../bin/optipng-bin');
 		var args = [
 			'-strip',
@@ -39,7 +38,7 @@ describe('OptiPNG', function() {
 			path.join(__dirname, 'fixtures', 'test.png')
 		];
 
-		execFile(binPath, args, function() {
+		execFile(binPath, args, function () {
 			var actual = fs.statSync('test/minified.png').size;
 			var original = fs.statSync('test/fixtures/test.png').size;
 			assert(actual < original);