Merge pull request #26 from joscha/minor-fixes

fixes minor code problems, such as missing semicolons, etc.

.jshintrc

@@ -0,0 +1,6 @@
+{
+  "curly": true,
+  "eqeqeq": true,
+  "undef": true,
+  "node": true
+}

bin/crx.js

@@ -13,7 +13,7 @@ var join = path.join;
 
 var cwd = process.cwd();
 
-program.version(pkg.version)
+program.version(pkg.version);
 // coming soon
 // .option("-x, --xml", "output autoupdate xml instead of extension ")
 
@@ -52,8 +52,8 @@ function keygen (dir, program) {
       }
 
       console.log('%s has been generated in %s', 'key.pem', dir);
-    })
-  })
+    });
+  });
 }
 
 function pack (dir, program) {

src/crx.js

@@ -1,3 +1,4 @@
+/* global require, process, Buffer, module */
 'use strict';
 
 var fs = require("fs");
@@ -181,7 +182,7 @@ ChromeExtension.prototype = {
       var allFiles = [];
 
       if (!selfie.loaded) {
-	throw new Error('crx.load needs to be called first in order to prepare the workspace.')
+	throw new Error('crx.load needs to be called first in order to prepare the workspace.');
       }
 
       // the callback is called many times
@@ -244,7 +245,7 @@ ChromeExtension.prototype = {
 
     var crx = new Buffer(length);
 
-    crx.write("Cr24" + Array(13).join("\x00"), "binary");
+    crx.write("Cr24" + new Array(13).join("\x00"), "binary");
 
     crx[4] = 2;
     crx.writeUInt32LE(keyLength, 8);
@@ -294,7 +295,7 @@ ChromeExtension.prototype = {
       throw new Error("No URL provided for update.xml.");
     }
 
-    return Buffer(
+    return new Buffer(
       "<?xml version='1.0' encoding='UTF-8'?>\n" +
       "<gupdate xmlns='http://www.google.com/update2/response' protocol='2.0'>\n" +
       "  <app appid='" + (this.appId || this.generateAppId()) + "'>\n" +

test/index.js

@@ -1,3 +1,6 @@
+/* global require, __dirname, Buffer */
+'use strict';
+
 var fs = require("fs");
 var test = require("tape");
 var ChromeExtension = require("../");