jshint

Makefile

@@ -34,4 +34,7 @@ test: test-tmp
 	@if test -e "node_modules/expresso/"; then ./node_modules/expresso/bin/expresso -I lib tests/${only}.test.js; else expresso -I lib tests/${only}.test.js; fi;
 endif
 
+lint:
+	./node_modules/.bin/jshint lib/*js --config=jshint.json
+
 .PHONY: test

lib/mapnik.js

@@ -13,7 +13,7 @@ exports = module.exports = mapnik;
   If not the same versions register_fonts may throw with:
      "TypeError: first argument must be a path to a directory of fonts"
 */
-if (!(mapnik.versions.node === process.versions.node)) {
+if (mapnik.versions.node != process.versions.node) {
     console.warn('node-mapnik warning: The running node version "' + process.versions.node + '" does not match the node version that node-mapnik was compiled against: "' + mapnik.versions.node + '", undefined behavior may result');
 }
 
@@ -119,7 +119,7 @@ exports.register_system_fonts = function() {
     // TODO mysys and cygwin
 
     var found = [];
-    for (idx in dirs) {
+    for (var idx in dirs) {
         var p = dirs[idx];
         if (path.existsSync(p)) {
                 if (mapnik.register_fonts(p, {recurse: true}))
@@ -140,5 +140,5 @@ exports.register_system_fonts = function() {
 exports.register_system_fonts();
 
 // make settings available
-exports['settings'] = _settings;
+exports.settings = _settings;
 

package.json

@@ -20,7 +20,8 @@
   "licenses"      : [{"type": "BSD"}],
   "dependencies"  : {},
   "devDependencies": {
-    "expresso": "0.8.x"
+    "expresso": "0.8.x",
+    "jshint"  : "0.2.3"
    },
   "bin"           : {
        "mapnik-inspect.js" : "./bin/mapnik-inspect.js",