From 40898c19a10bd074c9c50fb06bf2e9ae0ca25544 Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 15:06:11 -0700
Subject: [PATCH 1/9] enable JSHint's "bitwise" test

---
 .jshintrc                             | 2 +-
 rhino_modules/jsdoc/tag/dictionary.js | 6 +++++-
 rhino_modules/jsdoc/tag/type.js       | 2 +-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/.jshintrc b/.jshintrc
index c1260897..8c8a75fa 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -1,5 +1,5 @@
 {
-  "bitwise":      false,
+  "bitwise":      true,
   "curly":        false,
   "eqeqeq":       false,
   "forin":        false,
diff --git a/rhino_modules/jsdoc/tag/dictionary.js b/rhino_modules/jsdoc/tag/dictionary.js
index 4f3c1651..18b1492a 100644
--- a/rhino_modules/jsdoc/tag/dictionary.js
+++ b/rhino_modules/jsdoc/tag/dictionary.js
@@ -52,7 +52,11 @@ var dictionary = {
     
     /** @function */
     isNamespace: function(kind) {
-        return ( ~ _namespaces.indexOf(kind) );
+        if ( _namespaces.indexOf(kind) !== -1) {
+            return true;
+        }
+        
+        return false;
     },
     
     /** @function */
diff --git a/rhino_modules/jsdoc/tag/type.js b/rhino_modules/jsdoc/tag/type.js
index 256cc5d4..d799bad9 100644
--- a/rhino_modules/jsdoc/tag/type.js
+++ b/rhino_modules/jsdoc/tag/type.js
@@ -82,7 +82,7 @@ function parseVariable(type) {
 function parseTypes(type) {
 	var types = [];
 	
-	if ( ~type.indexOf('|') ) {
+	if ( type.indexOf('|') !== -1 ) {
 		// remove optional parens, like: { ( string | number ) }
 		// see: http://code.google.com/closure/compiler/docs/js-for-compiler.html#types
 		if ( /^\s*\(\s*(.+)\s*\)\s*$/.test(type) ) {

From 324af73dcfde8eb6176183f163b497ada5ed2cef Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 15:19:31 -0700
Subject: [PATCH 2/9] enable JSHint's "curly" test

---
 .jshintrc                                     |  2 +-
 rhino_modules/jsdoc/borrow.js                 |  4 +-
 rhino_modules/jsdoc/doclet.js                 |  4 +-
 rhino_modules/jsdoc/name.js                   |  8 +--
 rhino_modules/jsdoc/src/parser.js             | 26 +++++++---
 .../jsdoc/tag/dictionary/definitions.js       |  8 ++-
 rhino_modules/jsdoc/util/dumper.js            |  4 +-
 templates/default/publish.js                  | 52 +++++++++++++------
 8 files changed, 72 insertions(+), 36 deletions(-)

diff --git a/.jshintrc b/.jshintrc
index 8c8a75fa..08eb3b40 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -1,6 +1,6 @@
 {
   "bitwise":      true,
-  "curly":        false,
+  "curly":        true,
   "eqeqeq":       false,
   "forin":        false,
   "immed":        false,
diff --git a/rhino_modules/jsdoc/borrow.js b/rhino_modules/jsdoc/borrow.js
index 12054639..44dba578 100644
--- a/rhino_modules/jsdoc/borrow.js
+++ b/rhino_modules/jsdoc/borrow.js
@@ -32,8 +32,8 @@ exports.resolveBorrows = function(docs) {
                         asName = asName.replace(/^prototype\./, '#');
                         var parts = asName.split('#');
                         
-                        if (parts.length === 2) clone.scope = 'instance';
-                        else clone.scope = 'static';
+                        if (parts.length === 2) { clone.scope = 'instance'; }
+                        else { clone.scope = 'static'; }
 
                         asName = parts.pop();
                         clone.name = asName;
diff --git a/rhino_modules/jsdoc/doclet.js b/rhino_modules/jsdoc/doclet.js
index aae2f6e2..21c06a9a 100644
--- a/rhino_modules/jsdoc/doclet.js
+++ b/rhino_modules/jsdoc/doclet.js
@@ -126,7 +126,7 @@ exports.Doclet.prototype.setLongname = function(name) {
 */
 exports.Doclet.prototype.borrow = function(source, target) {
     var about = {from: source};
-    if (target) about.as = target;
+    if (target) { about.as = target; }
     
     if (!this.borrowed) {
         /**
@@ -203,7 +203,7 @@ exports.Doclet.prototype.setMeta = function(meta) {
         @namespace
      */
     this.meta.code = (this.meta.code || {});
-    if (meta.id) this.meta.code.id = meta.id;
+    if (meta.id) { this.meta.code.id = meta.id; }
     if (meta.code) {
         if (meta.code.name) {
             /** The name of the symbol in the source code. */
diff --git a/rhino_modules/jsdoc/name.js b/rhino_modules/jsdoc/name.js
index d5c7d064..815d98c1 100644
--- a/rhino_modules/jsdoc/name.js
+++ b/rhino_modules/jsdoc/name.js
@@ -79,7 +79,9 @@ exports.resolve = function(doclet) {
                 doclet.scope = puncToScope[RegExp.$1];
                 doclet.name = doclet.name.substr(1);
             }
-            else doclet.scope = 'static'; // default scope when none is provided
+            else {
+                doclet.scope = 'static'; // default scope when none is provided
+            }
          
             doclet.setLongname(doclet.memberof + scopeToPunc[doclet.scope] + doclet.name);
         }
@@ -166,8 +168,8 @@ exports.shorten = function(longname, forcedMemberof) {
         name = longname.substr(forcedMemberof.length);
         var parts = forcedMemberof.match(/^(.*?)([#.~]?)$/);
 
-        if (parts[1]) memberof = parts[1] || forcedMemberof;
-        if (parts[2]) scope = parts[2];
+        if (parts[1]) { memberof = parts[1] || forcedMemberof; }
+        if (parts[2]) { scope = parts[2]; }
     }
     else {
         var parts = longname?
diff --git a/rhino_modules/jsdoc/src/parser.js b/rhino_modules/jsdoc/src/parser.js
index 55253ab9..96969a99 100644
--- a/rhino_modules/jsdoc/src/parser.js
+++ b/rhino_modules/jsdoc/src/parser.js
@@ -199,7 +199,9 @@ exports.Parser.prototype.astnodeToMemberof = function(node) {
 
         id = 'astnode'+node.parent.hashCode();
         doclet = this.refs[id];
-        if (!doclet) return ''; // global?
+        if (!doclet) {
+            return ''; // global?
+        }
         return doclet.longname||doclet.name;
     }
 }
@@ -236,16 +238,22 @@ exports.Parser.prototype.resolveThis = function(node) {
             if (node.enclosingFunction){
                 return this.resolveThis(node.enclosingFunction/* memberof.doclet.meta.code.val */);
             }
-            else return ''; // TODO handle global this?
+            else {
+                return ''; // TODO handle global this?
+            }
         }
     }
     else if (node.parent) {
         var parent = node.parent;
-        if (parent.type === Token.COLON) parent = parent.parent; // go up one more
+        if (parent.type === Token.COLON) {
+            parent = parent.parent; // go up one more
+        }
 
         memberof.id = 'astnode'+parent.hashCode();
         memberof.doclet = this.refs[memberof.id];
-        if (!memberof.doclet) return ''; // global?
+        if (!memberof.doclet) {
+            return ''; // global?
+        }
 
         return memberof.doclet.longname||memberof.doclet.name;
     }
@@ -262,7 +270,9 @@ exports.Parser.prototype.resolvePropertyParent = function(node) {
 
     if (node.parent) {
         var parent = node.parent;
-        if (parent.type === Token.COLON) parent = parent.parent; // go up one more
+        if (parent.type === Token.COLON) {
+            parent = parent.parent; // go up one more
+        }
 
         memberof.id = 'astnode'+parent.hashCode();
         memberof.doclet = this.refs[memberof.id];
@@ -366,7 +376,9 @@ function visitNode(node) {
 
         var basename = getBasename(e.code.name);
 
-        if (basename !== 'this') e.code.funcscope = currentParser.resolveVar(node, basename);
+        if (basename !== 'this') {
+            e.code.funcscope = currentParser.resolveVar(node, basename);
+        }
     }
     else if (node.type === Token.COLON) { // assignment within an object literal
         e = {
@@ -553,7 +565,7 @@ function aboutNode(node) {
 function nodeToString(node) {
     var str;
 
-    if (!node) return;
+    if (!node) { return; }
 
     if (node.type === Token.GETPROP) {
         str = [nodeToString(node.target), node.property.string].join('.');
diff --git a/rhino_modules/jsdoc/tag/dictionary/definitions.js b/rhino_modules/jsdoc/tag/dictionary/definitions.js
index 82fe601f..ab5f0cf8 100644
--- a/rhino_modules/jsdoc/tag/dictionary/definitions.js
+++ b/rhino_modules/jsdoc/tag/dictionary/definitions.js
@@ -513,7 +513,9 @@ exports.defineTags = function(dictionary) {
         onTagged: function(doclet, tag) {
             if (tag.value && tag.value.type) {
                 doclet.type = tag.value.type;
-                if (doclet.kind === 'function') doclet.addTag('returns', tag.text); // for backwards compatibility we allow @type for functions to imply return type
+                if (doclet.kind === 'function') {
+                    doclet.addTag('returns', tag.text); // for backwards compatibility we allow @type for functions to imply return type
+                }
             }
         }
     });
@@ -601,7 +603,9 @@ function applyNamespace(docletOrNs, tag) {
         tag.value = app.jsdoc.name.applyNamespace(tag.value, docletOrNs);
     }
     else { // doclet
-        if (!docletOrNs.name) return; // error?
+        if (!docletOrNs.name) {
+            return; // error?
+        }
         
         //doclet.displayname = doclet.name;
         docletOrNs.longname = app.jsdoc.name.applyNamespace(docletOrNs.name, tag.title);
diff --git a/rhino_modules/jsdoc/util/dumper.js b/rhino_modules/jsdoc/util/dumper.js
index 1e13d2e0..9392401a 100644
--- a/rhino_modules/jsdoc/util/dumper.js
+++ b/rhino_modules/jsdoc/util/dumper.js
@@ -37,7 +37,7 @@ function pad(depth) {
  */
 function indent(openingBrace) {
     indentBy++;
-    if (openingBrace) output += openingBrace + '\n';
+    if (openingBrace) { output += openingBrace + '\n'; }
 }
 
 /**
@@ -51,7 +51,7 @@ function outdent(closingBrace) {
     indentBy--;
     output = output.replace(/,\n$/, '\n'); // trim trailing comma
     if (closingBrace === false) { output = output.replace(/\n$/, ''); }
-    else if (closingBrace) output += pad(indentBy) + closingBrace + ',\n';
+    else if (closingBrace) { output += pad(indentBy) + closingBrace + ',\n'; }
 }
 
 var seen = [];
diff --git a/templates/default/publish.js b/templates/default/publish.js
index 794eb3d3..fd40ea8a 100644
--- a/templates/default/publish.js
+++ b/templates/default/publish.js
@@ -53,7 +53,7 @@
             while (doc = doc.memberof) {
                 doc = find({longname: doc});
                 if (doc) { doc = doc[0]; }
-                if (!doc) break;
+                if (!doc) { break; }
                 ancestors.unshift( linkto(doc.longname, (scopeToPunc[doc.scope] || '') + doc.name) );
             }
             if (ancestors.length) {
@@ -109,11 +109,15 @@
             }
             
             if (f.scope && f.scope !== 'instance' && f.scope !== 'global') {
-                if (f.kind == 'function' || f.kind == 'member' || f.kind == 'constant') attribs.push(f.scope);
+                if (f.kind == 'function' || f.kind == 'member' || f.kind == 'constant') {
+                    attribs.push(f.scope);
+                }
             }
             
             if (f.readonly === true) {
-                if (f.kind == 'member') attribs.push('readonly');
+                if (f.kind == 'member') {
+                    attribs.push('readonly');
+                }
             }
             
             if (f.kind === 'constant') {
@@ -246,7 +250,9 @@
         if (moduleNames.length) {
             nav += '<h3>Modules</h3><ul>';
             moduleNames.forEach(function(m) {
-                if ( !hasOwnProp.call(seen, m.longname) ) nav += '<li>'+linkto(m.longname, m.name)+'</li>';
+                if ( !hasOwnProp.call(seen, m.longname) ) {
+                    nav += '<li>'+linkto(m.longname, m.name)+'</li>';
+                }
                 seen[m.longname] = true;
             });
             
@@ -260,7 +266,9 @@
         if (externalNames.length) {
             nav += '<h3>Externals</h3><ul>';
             externalNames.forEach(function(e) {
-                if ( !hasOwnProp.call(seen, e.longname) ) nav += '<li>'+linkto( e.longname, e.name.replace(/(^"|"$)/g, '') )+'</li>';
+                if ( !hasOwnProp.call(seen, e.longname) ) {
+                    nav += '<li>'+linkto( e.longname, e.name.replace(/(^"|"$)/g, '') )+'</li>';
+                }
                 seen[e.longname] = true;
             });
             
@@ -284,7 +292,9 @@
                     nav += '<h3>Classes</h3><ul>';
                     moduleClasses = -1;
                 }
-                if ( !hasOwnProp.call(seen, c.longname) ) nav += '<li>'+linkto(c.longname, c.name)+'</li>';
+                if ( !hasOwnProp.call(seen, c.longname) ) {
+                    nav += '<li>'+linkto(c.longname, c.name)+'</li>';
+                }
                 seen[c.longname] = true;
             });
             
@@ -298,7 +308,9 @@
         if (namespaceNames.length) {
             nav += '<h3>Namespaces</h3><ul>';
             namespaceNames.forEach(function(n) {
-                if ( !hasOwnProp.call(seen, n.longname) ) nav += '<li>'+linkto(n.longname, n.name)+'</li>';
+                if ( !hasOwnProp.call(seen, n.longname) ) {
+                    nav += '<li>'+linkto(n.longname, n.name)+'</li>';
+                }
                 seen[n.longname] = true;
             });
             
@@ -309,7 +321,9 @@
 //         if (constantNames.length) {
 //             nav += '<h3>Constants</h3><ul>';
 //             constantNames.forEach(function(c) {
-//                 if ( !hasOwnProp.call(seen, c.longname) ) nav += '<li>'+linkto(c.longname, c.name)+'</li>';
+//                 if ( !hasOwnProp.call(seen, c.longname) ) {
+//                     nav += '<li>'+linkto(c.longname, c.name)+'</li>';
+//                 }
 //                 seen[c.longname] = true;
 //             });
 //             
@@ -323,7 +337,9 @@
         if (mixinNames.length) {
             nav += '<h3>Mixins</h3><ul>';
             mixinNames.forEach(function(m) {
-                if ( !hasOwnProp.call(seen, m.longname) ) nav += '<li>'+linkto(m.longname, m.name)+'</li>';
+                if ( !hasOwnProp.call(seen, m.longname) ) {
+                    nav += '<li>'+linkto(m.longname, m.name)+'</li>';
+                }
                 seen[m.longname] = true;
             });
             
@@ -347,7 +363,9 @@
             nav += '<h3>Global</h3><ul>';
             
             globalNames.forEach(function(g) {
-                if ( g.kind !== 'typedef' && !hasOwnProp.call(seen, g.longname) ) nav += '<li>'+linkto(g.longname, g.name)+'</li>';
+                if ( g.kind !== 'typedef' && !hasOwnProp.call(seen, g.longname) ) {
+                    nav += '<li>'+linkto(g.longname, g.name)+'</li>';
+                }
                 seen[g.longname] = true;
             });
             
@@ -364,25 +382,25 @@
 
         for (var longname in helper.longnameToUrl) {
             var classes = find({kind: 'class', longname: longname});
-            if (classes.length) generate('Class: '+classes[0].name, classes, helper.longnameToUrl[longname]);
+            if (classes.length) { generate('Class: '+classes[0].name, classes, helper.longnameToUrl[longname]); }
         
             var modules = find({kind: 'module', longname: longname});
-            if (modules.length) generate('Module: '+modules[0].name, modules, helper.longnameToUrl[longname]);
+            if (modules.length) { generate('Module: '+modules[0].name, modules, helper.longnameToUrl[longname]); }
             
             var namespaces = find({kind: 'namespace', longname: longname});
-            if (namespaces.length) generate('Namespace: '+namespaces[0].name, namespaces, helper.longnameToUrl[longname]);        
+            if (namespaces.length) { generate('Namespace: '+namespaces[0].name, namespaces, helper.longnameToUrl[longname]); }
             
 //             var constants = find({kind: 'constant', longname: longname});
-//             if (constants.length) generate('Constant: '+constants[0].name, constants, helper.longnameToUrl[longname]);        
+//             if (constants.length) { generate('Constant: '+constants[0].name, constants, helper.longnameToUrl[longname]); }
 
             var mixins = find({kind: 'mixin', longname: longname});
-            if (mixins.length) generate('Mixin: '+mixins[0].name, mixins, helper.longnameToUrl[longname]);        
+            if (mixins.length) { generate('Mixin: '+mixins[0].name, mixins, helper.longnameToUrl[longname]); }
         
             var externals = find({kind: 'external', longname: longname});
-            if (externals.length) generate('External: '+externals[0].name, externals, helper.longnameToUrl[longname]);
+            if (externals.length) { generate('External: '+externals[0].name, externals, helper.longnameToUrl[longname]); }
         }
 
-        if (globals.length) generate('Global', [{kind: 'globalobj'}], 'global.html');
+        if (globals.length) { generate('Global', [{kind: 'globalobj'}], 'global.html'); }
         
         // index page displays information from package.json and lists files
         var files = find({kind: 'file'}),

From 769a71298e0daa1c331e3c8e83101cb7b0c53a78 Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 15:59:33 -0700
Subject: [PATCH 3/9] enable JSHint's "forin" test

---
 .jshintrc                                |  2 +-
 jsdoc.js                                 |  9 +++--
 rhino_modules/jsdoc/augment.js           | 26 ++++++++-----
 rhino_modules/jsdoc/tutorial/resolver.js | 47 +++++++++++++-----------
 templates/default/publish.js             | 26 +++++++------
 5 files changed, 62 insertions(+), 48 deletions(-)

diff --git a/.jshintrc b/.jshintrc
index 08eb3b40..737dd285 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -2,7 +2,7 @@
   "bitwise":      true,
   "curly":        true,
   "eqeqeq":       false,
-  "forin":        false,
+  "forin":        true,
   "immed":        false,
   "latedef":      false,
   "newcap":       false,
diff --git a/jsdoc.js b/jsdoc.js
index e2f3d19c..66dada0a 100644
--- a/jsdoc.js
+++ b/jsdoc.js
@@ -8,6 +8,8 @@
 
 //~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~//
 
+var hasOwnProp = Object.prototype.hasOwnProperty;
+
 /** Data representing the environment in which this app is running.
     @namespace
 */
@@ -151,7 +153,9 @@ function installPlugins(plugins, p) {
         //...register event handlers
         if (plugin.handlers) {
             for (var eventName in plugin.handlers) {
-                parser.on(eventName, plugin.handlers[eventName]);
+                if ( hasOwnProp.call(plugin.handlers, eventName) ) {
+                    parser.on(eventName, plugin.handlers[eventName]);
+                }
             }
         }
 
@@ -168,8 +172,7 @@ function installPlugins(plugins, p) {
 }
 
 function indexAll(docs) {
-    var lookupTable = {},
-        hasOwnProp = Object.prototype.hasOwnProperty;
+    var lookupTable = {};
 
     docs.forEach(function(doc) {
         if ( !hasOwnProp.call(lookupTable, doc.longname) ) {
diff --git a/rhino_modules/jsdoc/augment.js b/rhino_modules/jsdoc/augment.js
index bd9d1ecc..1923e8aa 100644
--- a/rhino_modules/jsdoc/augment.js
+++ b/rhino_modules/jsdoc/augment.js
@@ -23,14 +23,16 @@ var doop = require("jsdoc/util/doop").doop;
     function mapDependencies(index) {
         var doclets, doc, len, dependencies = {};
         for (var name in index) {
-            doclets = index[name];
-            for (var i=0, ii=doclets.length; i<ii; ++i) {
-                doc = doclets[i];
-                if (doc.kind === "class") {
-                    dependencies[name] = {};
-                    len = doc.augments && doc.augments.length || 0;
-                    for (var j=0; j<len; ++j) {
-                        dependencies[name][doc.augments[j]] = true;
+            if ( hasOwnProp.call(index, name) ) {
+                doclets = index[name];
+                for (var i=0, ii=doclets.length; i<ii; ++i) {
+                    doc = doclets[i];
+                    if (doc.kind === "class") {
+                        dependencies[name] = {};
+                        len = doc.augments && doc.augments.length || 0;
+                        for (var j=0; j<len; ++j) {
+                            dependencies[name][doc.augments[j]] = true;
+                        }
                     }
                 }
             }
@@ -83,7 +85,9 @@ var doop = require("jsdoc/util/doop").doop;
     Sorter.prototype = {
         sort: function() {
             for (var key in this.dependencies) {
-                this.visit(key);
+                if ( hasOwnProp.call(this.dependencies, key) ) {
+                    this.visit(key);
+                }
             }
             return this.sorted;
         },
@@ -94,7 +98,9 @@ var doop = require("jsdoc/util/doop").doop;
                     throw new Error("Missing dependency: " + key);
                 }
                 for (var path in this.dependencies[key]) {
-                    this.visit(path);
+                    if ( hasOwnProp.call(this.dependencies[key], path) ) {
+                        this.visit(path);
+                    }
                 }
                 this.sorted.push(key);
             }
diff --git a/rhino_modules/jsdoc/tutorial/resolver.js b/rhino_modules/jsdoc/tutorial/resolver.js
index 1ee97869..0f0d1943 100644
--- a/rhino_modules/jsdoc/tutorial/resolver.js
+++ b/rhino_modules/jsdoc/tutorial/resolver.js
@@ -10,6 +10,7 @@
 
 var tutorial = require('jsdoc/tutorial'),
     fs = require('fs'),
+    hasOwnProp = Object.prototype.hasOwnProperty,
     conf = {},
     tutorials = {},
     finder = /^(.*)\.(x(?:ht)?ml|html?|md|markdown|js(?:on)?)$/i;
@@ -97,32 +98,34 @@ exports.resolve = function() {
     var item,
         current;
     for (var name in conf) {
-        // should we be restrictive here?
-        // what is someone just wants to keep sample sources in same directory with tutorials?
-        // I've decided to leave such cases alone
-        if (!(name in tutorials)) {
-            continue;
-        }
+        if ( hasOwnProp.call(conf, name) ) {
+            // should we be restrictive here?
+            // what is someone just wants to keep sample sources in same directory with tutorials?
+            // I've decided to leave such cases alone
+            if (!(name in tutorials)) {
+                continue;
+            }
 
-        item = conf[name];
-        current = tutorials[name]
+            item = conf[name];
+            current = tutorials[name]
 
-        // set title
-        if (item.title) {
-            current.title = item.title;
-        }
+            // set title
+            if (item.title) {
+                current.title = item.title;
+            }
 
-        // add children
-        if (item.children) {
-            item.children.forEach(function(child) {
-                // I really didn't want to throw you an exception in most cases
-                // but now, user, you pissed me off ;)
-                if (!(child in tutorials)) {
-                    throw new Error("Missing child tutorial: " + child);
-                }
+            // add children
+            if (item.children) {
+                item.children.forEach(function(child) {
+                    // I really didn't want to throw you an exception in most cases
+                    // but now, user, you pissed me off ;)
+                    if (!(child in tutorials)) {
+                        throw new Error("Missing child tutorial: " + child);
+                    }
 
-                tutorials[child].setParent(current);
-            });
+                    tutorials[child].setParent(current);
+                });
+            }
         }
     }
 };
diff --git a/templates/default/publish.js b/templates/default/publish.js
index fd40ea8a..2f5b2283 100644
--- a/templates/default/publish.js
+++ b/templates/default/publish.js
@@ -381,23 +381,25 @@
         view.nav = nav;
 
         for (var longname in helper.longnameToUrl) {
-            var classes = find({kind: 'class', longname: longname});
-            if (classes.length) { generate('Class: '+classes[0].name, classes, helper.longnameToUrl[longname]); }
+            if ( hasOwnProp.call(helper.longnameToUrl, longname) ) {
+                var classes = find({kind: 'class', longname: longname});
+                if (classes.length) { generate('Class: '+classes[0].name, classes, helper.longnameToUrl[longname]); }
         
-            var modules = find({kind: 'module', longname: longname});
-            if (modules.length) { generate('Module: '+modules[0].name, modules, helper.longnameToUrl[longname]); }
+                var modules = find({kind: 'module', longname: longname});
+                if (modules.length) { generate('Module: '+modules[0].name, modules, helper.longnameToUrl[longname]); }
             
-            var namespaces = find({kind: 'namespace', longname: longname});
-            if (namespaces.length) { generate('Namespace: '+namespaces[0].name, namespaces, helper.longnameToUrl[longname]); }
+                var namespaces = find({kind: 'namespace', longname: longname});
+                if (namespaces.length) { generate('Namespace: '+namespaces[0].name, namespaces, helper.longnameToUrl[longname]); }
             
-//             var constants = find({kind: 'constant', longname: longname});
-//             if (constants.length) { generate('Constant: '+constants[0].name, constants, helper.longnameToUrl[longname]); }
+//                var constants = find({kind: 'constant', longname: longname});
+//                if (constants.length) { generate('Constant: '+constants[0].name, constants, helper.longnameToUrl[longname]); }
 
-            var mixins = find({kind: 'mixin', longname: longname});
-            if (mixins.length) { generate('Mixin: '+mixins[0].name, mixins, helper.longnameToUrl[longname]); }
+                var mixins = find({kind: 'mixin', longname: longname});
+                if (mixins.length) { generate('Mixin: '+mixins[0].name, mixins, helper.longnameToUrl[longname]); }
         
-            var externals = find({kind: 'external', longname: longname});
-            if (externals.length) { generate('External: '+externals[0].name, externals, helper.longnameToUrl[longname]); }
+                var externals = find({kind: 'external', longname: longname});
+                if (externals.length) { generate('External: '+externals[0].name, externals, helper.longnameToUrl[longname]); }
+            }
         }
 
         if (globals.length) { generate('Global', [{kind: 'globalobj'}], 'global.html'); }

From dabb0b4410b1b90cc3dd5adfa8e9cf8c40197e76 Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 16:01:55 -0700
Subject: [PATCH 4/9] enable JSHint's "immed" test

---
 .jshintrc                      | 2 +-
 rhino_modules/jsdoc/augment.js | 2 +-
 templates/default/publish.js   | 2 +-
 templates/haruki/publish.js    | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/.jshintrc b/.jshintrc
index 737dd285..a0595ff4 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -3,7 +3,7 @@
   "curly":        true,
   "eqeqeq":       false,
   "forin":        true,
-  "immed":        false,
+  "immed":        true,
   "latedef":      false,
   "newcap":       false,
   "noarg":        false,
diff --git a/rhino_modules/jsdoc/augment.js b/rhino_modules/jsdoc/augment.js
index 1923e8aa..0f6a6d3c 100644
--- a/rhino_modules/jsdoc/augment.js
+++ b/rhino_modules/jsdoc/augment.js
@@ -113,4 +113,4 @@ var doop = require("jsdoc/util/doop").doop;
     }
 
     
-})();
+}());
diff --git a/templates/default/publish.js b/templates/default/publish.js
index 2f5b2283..03e715a7 100644
--- a/templates/default/publish.js
+++ b/templates/default/publish.js
@@ -465,4 +465,4 @@
         return '<a href="'+url+'">'+hash+'</a>';
     }
     
-})();
+}());
diff --git a/templates/haruki/publish.js b/templates/haruki/publish.js
index 42d4ccc7..d283f60b 100644
--- a/templates/haruki/publish.js
+++ b/templates/haruki/publish.js
@@ -209,5 +209,5 @@
         });
     }
 
-})();
+}());
 

From c60f68286f83af06887e1c5099939c54f60828c2 Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 16:05:08 -0700
Subject: [PATCH 5/9] enable JSHint's "newcap" test

---
 .jshintrc                             | 2 +-
 jsdoc.js                              | 4 ++--
 rhino_modules/jsdoc/tag/dictionary.js | 8 +++++---
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/.jshintrc b/.jshintrc
index a0595ff4..d66160e9 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -5,7 +5,7 @@
   "forin":        true,
   "immed":        true,
   "latedef":      false,
-  "newcap":       false,
+  "newcap":       true,
   "noarg":        false,
   "noempty":      false,
   "nonew":        false,
diff --git a/jsdoc.js b/jsdoc.js
index 66dada0a..871decd6 100644
--- a/jsdoc.js
+++ b/jsdoc.js
@@ -263,8 +263,8 @@ function main() {
         }
         
         if (/(\bREADME|\.md)$/i.test(env.opts._[i])) {
-            var readme = require('jsdoc/readme');
-            env.opts.readme = new readme(env.opts._[i]).html;
+            var Readme = require('jsdoc/readme');
+            env.opts.readme = new Readme(env.opts._[i]).html;
             env.opts._.splice(i--, 1);
         }
     }
diff --git a/rhino_modules/jsdoc/tag/dictionary.js b/rhino_modules/jsdoc/tag/dictionary.js
index 18b1492a..1182ca6b 100644
--- a/rhino_modules/jsdoc/tag/dictionary.js
+++ b/rhino_modules/jsdoc/tag/dictionary.js
@@ -9,7 +9,8 @@ var _synonyms = {},
     _namespaces = [],
     hasOwnProp = Object.prototype.hasOwnProperty;
 
-function _TagDefinition(title, etc) {
+/** @private */
+function TagDefinition(title, etc) {
     etc = etc || {};
     
     this.title = dictionary.normalise(title);
@@ -21,7 +22,8 @@ function _TagDefinition(title, etc) {
     }
 }
 
-_TagDefinition.prototype.synonym = function(synonymName) {
+/** @private */
+TagDefinition.prototype.synonym = function(synonymName) {
     _synonyms[synonymName.toLowerCase()] = this.title;
     return this; // chainable
 }
@@ -30,7 +32,7 @@ _TagDefinition.prototype.synonym = function(synonymName) {
 var dictionary = {
     /** @function */
     defineTag: function(title, opts) {
-        _definitions[title] = new _TagDefinition(title, opts);
+        _definitions[title] = new TagDefinition(title, opts);
         
         if (opts.isNamespace) {
             _namespaces.push(title);

From 3f13364ec2113d644d751050e13a854dd61d6c0d Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 16:06:01 -0700
Subject: [PATCH 6/9] enable JSHint's "noarg" test (no fixes necessary)

---
 .jshintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.jshintrc b/.jshintrc
index d66160e9..b8763e4d 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -6,7 +6,7 @@
   "immed":        true,
   "latedef":      false,
   "newcap":       true,
-  "noarg":        false,
+  "noarg":        true,
   "noempty":      false,
   "nonew":        false,
   "plusplus":     false,

From 24b3c1114f383604f7348f7663483da7ed6c822e Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 16:07:19 -0700
Subject: [PATCH 7/9] enable JSHint's "nonew" test (no fixes necessary)

---
 .jshintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.jshintrc b/.jshintrc
index b8763e4d..b2c0477b 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -8,7 +8,7 @@
   "newcap":       true,
   "noarg":        true,
   "noempty":      false,
-  "nonew":        false,
+  "nonew":        true,
   "plusplus":     false,
   "regexp":       false,
   "undef":        false,

From 1951ea3821c13cd52b539a5c5d850ee69c5e2243 Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 16:22:08 -0700
Subject: [PATCH 8/9] enable JSHint's "undef" test

---
 .jshintrc                                         | 2 +-
 Jakefile.js                                       | 1 +
 jsdoc.js                                          | 1 +
 plugins/markdown.js                               | 1 +
 rhino_modules/fs.js                               | 1 +
 rhino_modules/jsdoc/augment.js                    | 2 +-
 rhino_modules/jsdoc/doclet.js                     | 4 +++-
 rhino_modules/jsdoc/name.js                       | 1 +
 rhino_modules/jsdoc/readme.js                     | 1 +
 rhino_modules/jsdoc/src/handlers.js               | 1 +
 rhino_modules/jsdoc/src/parser.js                 | 7 +++++--
 rhino_modules/jsdoc/tag.js                        | 1 +
 rhino_modules/jsdoc/tag/dictionary.js             | 2 +-
 rhino_modules/jsdoc/tag/dictionary/definitions.js | 1 +
 rhino_modules/jsdoc/tag/validator.js              | 1 +
 rhino_modules/jsdoc/tutorial/resolver.js          | 1 +
 rhino_modules/jsdoc/util/templateHelper.js        | 2 +-
 templates/default/publish.js                      | 3 ++-
 templates/haruki/publish.js                       | 1 +
 19 files changed, 26 insertions(+), 8 deletions(-)

diff --git a/.jshintrc b/.jshintrc
index b2c0477b..f9ad2431 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -11,7 +11,7 @@
   "nonew":        true,
   "plusplus":     false,
   "regexp":       false,
-  "undef":        false,
+  "undef":        true,
   "strict":       false,
   "trailing":     false,
 
diff --git a/Jakefile.js b/Jakefile.js
index a168e7a3..124b8ea3 100644
--- a/Jakefile.js
+++ b/Jakefile.js
@@ -1,3 +1,4 @@
+/*global desc: true, fail: true, Mustache: true, task: true */
 // see: https://github.com/mde/jake
 
 desc('Updating package.json revision.');
diff --git a/jsdoc.js b/jsdoc.js
index 871decd6..31d6f622 100644
--- a/jsdoc.js
+++ b/jsdoc.js
@@ -1,3 +1,4 @@
+/*global app: true, args: true, env: true, publish: true */
 /**
  * @project jsdoc
  * @author Michael Mathews <micmath@gmail.com>
diff --git a/plugins/markdown.js b/plugins/markdown.js
index d6d4f40d..a4736628 100644
--- a/plugins/markdown.js
+++ b/plugins/markdown.js
@@ -1,3 +1,4 @@
+/*global env: true */
 /**
     @overview Translate doclet descriptions from MarkDown into HTML.
     @module plugins/markdown
diff --git a/rhino_modules/fs.js b/rhino_modules/fs.js
index 98fb7a1c..8d085f79 100644
--- a/rhino_modules/fs.js
+++ b/rhino_modules/fs.js
@@ -1,3 +1,4 @@
+/*global Packages: true */
 exports.readFileSync = function(filename, encoding) {
     encoding = encoding || 'utf-8';
 
diff --git a/rhino_modules/jsdoc/augment.js b/rhino_modules/jsdoc/augment.js
index 0f6a6d3c..bff28a1e 100644
--- a/rhino_modules/jsdoc/augment.js
+++ b/rhino_modules/jsdoc/augment.js
@@ -42,7 +42,7 @@ var doop = require("jsdoc/util/doop").doop;
 
     function getAdditions(doclets, docs) {
         var additions = [];
-        var doc, parents, members, member;
+        var doc, parents, members, member, parts;
         for (var i=0, ii=doclets.length; i<ii; ++i) {
             doc = doclets[i];
             parents = doc.augments;
diff --git a/rhino_modules/jsdoc/doclet.js b/rhino_modules/jsdoc/doclet.js
index 21c06a9a..8fb82a13 100644
--- a/rhino_modules/jsdoc/doclet.js
+++ b/rhino_modules/jsdoc/doclet.js
@@ -300,7 +300,9 @@ function fixDescription(docletSrc) {
 }
 
 function split(docletSrc) {
-	var tagSrcs = [];
+    var tagSrcs = [],
+        tagText,
+        tagTitle;
           
 	// split out the basic tags, keep surrounding whitespace
 	// like: @tagTitle tagBody
diff --git a/rhino_modules/jsdoc/name.js b/rhino_modules/jsdoc/name.js
index 815d98c1..6d8161ea 100644
--- a/rhino_modules/jsdoc/name.js
+++ b/rhino_modules/jsdoc/name.js
@@ -1,3 +1,4 @@
+/*global Packages: true */
 /**
     A collection of functions relating to JSDoc symbol name manipulation.
     @module jsdoc/name
diff --git a/rhino_modules/jsdoc/readme.js b/rhino_modules/jsdoc/readme.js
index c95496a1..bbd98a85 100644
--- a/rhino_modules/jsdoc/readme.js
+++ b/rhino_modules/jsdoc/readme.js
@@ -1,3 +1,4 @@
+/*global env: true */
 /**
     @overview
     @author Michael Mathews <micmath@gmail.com>
diff --git a/rhino_modules/jsdoc/src/handlers.js b/rhino_modules/jsdoc/src/handlers.js
index 6f5c3c32..06ef880a 100644
--- a/rhino_modules/jsdoc/src/handlers.js
+++ b/rhino_modules/jsdoc/src/handlers.js
@@ -146,6 +146,7 @@ exports.attachTo = function(parser) {
     });
 
     function addDoclet(newDoclet) {
+        var e;
         if (newDoclet) {
             e = { doclet: newDoclet };
             this.fire('newDoclet', e);
diff --git a/rhino_modules/jsdoc/src/parser.js b/rhino_modules/jsdoc/src/parser.js
index 96969a99..036458ab 100644
--- a/rhino_modules/jsdoc/src/parser.js
+++ b/rhino_modules/jsdoc/src/parser.js
@@ -1,3 +1,4 @@
+/*global Packages: true */
 /**
  * @module jsdoc/src/parser
  * @requires common/util
@@ -47,7 +48,9 @@ require('common/util').mixin(exports.Parser.prototype, require('common/events'))
 exports.Parser.prototype.parse = function(sourceFiles, encoding) {
     const SCHEMA = 'javascript:';
     var sourceCode = '',
-        filename = '';
+        filename = '',
+        i,
+        leni;
 
     if (typeof sourceFiles === 'string') { sourceFiles = [sourceFiles]; }
 
@@ -495,7 +498,7 @@ function parserFactory() {
  * @memberof module:src/parser.Parser
  */
 function aboutNode(node) {
-    about = {};
+    var about = {};
 
     if (node.type == Token.FUNCTION || node.type == tkn.NAMEDFUNCTIONSTATEMENT) {
         about.name = node.type == tkn.NAMEDFUNCTIONSTATEMENT? '' : '' + node.name;
diff --git a/rhino_modules/jsdoc/tag.js b/rhino_modules/jsdoc/tag.js
index db02bcc3..28251201 100644
--- a/rhino_modules/jsdoc/tag.js
+++ b/rhino_modules/jsdoc/tag.js
@@ -1,3 +1,4 @@
+/*global env: true */
 /**
     @overview
     @author Michael Mathews <micmath@gmail.com>
diff --git a/rhino_modules/jsdoc/tag/dictionary.js b/rhino_modules/jsdoc/tag/dictionary.js
index 1182ca6b..61cfe122 100644
--- a/rhino_modules/jsdoc/tag/dictionary.js
+++ b/rhino_modules/jsdoc/tag/dictionary.js
@@ -63,7 +63,7 @@ var dictionary = {
     
     /** @function */
     normalise: function(title) {
-        canonicalName = title.toLowerCase();
+        var canonicalName = title.toLowerCase();
             
         if ( hasOwnProp.call(_synonyms, canonicalName) ) {
             return _synonyms[canonicalName];
diff --git a/rhino_modules/jsdoc/tag/dictionary/definitions.js b/rhino_modules/jsdoc/tag/dictionary/definitions.js
index ab5f0cf8..239eebef 100644
--- a/rhino_modules/jsdoc/tag/dictionary/definitions.js
+++ b/rhino_modules/jsdoc/tag/dictionary/definitions.js
@@ -1,3 +1,4 @@
+/*global app: true, env: true */
 /**
 	Define tags that are known in JSDoc.
 	@module jsdoc/tag/dictionary/definitions
diff --git a/rhino_modules/jsdoc/tag/validator.js b/rhino_modules/jsdoc/tag/validator.js
index cc02568d..bb4560cc 100644
--- a/rhino_modules/jsdoc/tag/validator.js
+++ b/rhino_modules/jsdoc/tag/validator.js
@@ -1,3 +1,4 @@
+/*global env: true */
 /**
 	@module jsdoc/tag/validator
 	@requires jsdoc/tag/dictionary
diff --git a/rhino_modules/jsdoc/tutorial/resolver.js b/rhino_modules/jsdoc/tutorial/resolver.js
index 0f0d1943..e587e64f 100644
--- a/rhino_modules/jsdoc/tutorial/resolver.js
+++ b/rhino_modules/jsdoc/tutorial/resolver.js
@@ -45,6 +45,7 @@ exports.load = function(path) {
     var match,
         type,
         name,
+        content,
         current,
         files = fs.ls(path);
 
diff --git a/rhino_modules/jsdoc/util/templateHelper.js b/rhino_modules/jsdoc/util/templateHelper.js
index 4303961a..c32a20de 100644
--- a/rhino_modules/jsdoc/util/templateHelper.js
+++ b/rhino_modules/jsdoc/util/templateHelper.js
@@ -131,7 +131,7 @@ exports.setTutorials = function(root) {
     tutorials = root;
 };
 
-exports.toTutorial = toTutorial = function(tutorial, content) {
+var toTutorial = exports.toTutorial = function(tutorial, content) {
     if (!tutorial) {
         throw new Error('Missing required parameter: tutorial');
     }
diff --git a/templates/default/publish.js b/templates/default/publish.js
index 03e715a7..d8a2e375 100644
--- a/templates/default/publish.js
+++ b/templates/default/publish.js
@@ -1,3 +1,4 @@
+/*global env: true, publish: true */
 (function() {
 
     var template = require('jsdoc/template'),
@@ -14,7 +15,7 @@
      */
     publish = function(data, opts, tutorials) {
         var defaultTemplatePath = 'templates/default';
-        var templatePath = (opts.template) ? opts.template : defaultTemplate;
+        var templatePath = (opts.template) ? opts.template : defaultTemplatePath;
         var out = '',
             view = new template.Template(env.dirname + '/' + templatePath + '/tmpl');
         
diff --git a/templates/haruki/publish.js b/templates/haruki/publish.js
index d283f60b..fd8d5503 100644
--- a/templates/haruki/publish.js
+++ b/templates/haruki/publish.js
@@ -1,3 +1,4 @@
+/*global publish: true */
 /**
     @overview Builds a tree-like JSON string from the doclet data.
     @version 0.0.1

From ce287745f15f420ec31006e6435e4b4505bca0ee Mon Sep 17 00:00:00 2001
From: Jeff Williams <jeffrey.l.williams@gmail.com>
Date: Wed, 4 Jul 2012 16:27:06 -0700
Subject: [PATCH 9/9] enable JSDoc's "trailing" test

---
 .jshintrc                                         | 2 +-
 jsdoc.js                                          | 2 +-
 rhino_modules/jsdoc/config.js                     | 2 +-
 rhino_modules/jsdoc/doclet.js                     | 2 +-
 rhino_modules/jsdoc/name.js                       | 2 +-
 rhino_modules/jsdoc/package.js                    | 4 ++--
 rhino_modules/jsdoc/src/filter.js                 | 4 ++--
 rhino_modules/jsdoc/tag/dictionary/definitions.js | 8 ++++----
 rhino_modules/jsdoc/util/doop.js                  | 4 ++--
 templates/default/publish.js                      | 6 +++---
 templates/haruki/publish.js                       | 2 +-
 11 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/.jshintrc b/.jshintrc
index f9ad2431..44926e14 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -13,7 +13,7 @@
   "regexp":       false,
   "undef":        true,
   "strict":       false,
-  "trailing":     false,
+  "trailing":     true,
 
   "asi":          true,
   "boss":         true,
diff --git a/jsdoc.js b/jsdoc.js
index 31d6f622..8ac91a39 100644
--- a/jsdoc.js
+++ b/jsdoc.js
@@ -33,7 +33,7 @@ env = {
     */
     conf: {},
 
-    /** 
+    /**
         The absolute path to the base directory of the jsdoc application.
         @type string
     */
diff --git a/rhino_modules/jsdoc/config.js b/rhino_modules/jsdoc/config.js
index e8f44e91..473da480 100644
--- a/rhino_modules/jsdoc/config.js
+++ b/rhino_modules/jsdoc/config.js
@@ -27,7 +27,7 @@ module.exports = Config;
     @class
     @classdesc Represents a JSDoc application configuration.
     @param {string} [json] - The contents of config.json.
- */ 
+ */
 function Config(json) {
     var json = JSON.parse( (json || "{}") );
     
diff --git a/rhino_modules/jsdoc/doclet.js b/rhino_modules/jsdoc/doclet.js
index 8fb82a13..af46b669 100644
--- a/rhino_modules/jsdoc/doclet.js
+++ b/rhino_modules/jsdoc/doclet.js
@@ -48,7 +48,7 @@ exports.Doclet = function(docletSrc, meta) {
 exports.Doclet.prototype.postProcess = function() {
     if (!this.preserveName) { jsdoc.name.resolve(this); }
     if (this.name && !this.longname) {
-        this.setLongname(this.name);  
+        this.setLongname(this.name);
     }
     if (this.memberof === '') {
         delete(this.memberof);
diff --git a/rhino_modules/jsdoc/name.js b/rhino_modules/jsdoc/name.js
index 6d8161ea..c5dc8b38 100644
--- a/rhino_modules/jsdoc/name.js
+++ b/rhino_modules/jsdoc/name.js
@@ -142,7 +142,7 @@ exports.applyNamespace = function(longname, ns) {
  */
 exports.shorten = function(longname, forcedMemberof) {
     // quoted strings in a longname are atomic, convert to tokens
-    var atoms = [], token; 
+    var atoms = [], token;
     
     // handle quoted names like foo["bar"]
     longname = longname.replace(/(\[?".+?"\]?)/g, function($) {
diff --git a/rhino_modules/jsdoc/package.js b/rhino_modules/jsdoc/package.js
index ef4b8125..3c645817 100644
--- a/rhino_modules/jsdoc/package.js
+++ b/rhino_modules/jsdoc/package.js
@@ -47,14 +47,14 @@ exports.Package = function(json) {
     */
     this.description = json.description;
     
-    /** 
+    /**
         The hash summary of the source file.
         @type {string}
         @since 3.2.0
     */
     this.version = json.version;
     
-    /** 
+    /**
      * The licenses of this package.
      * @type {Array<Object>}
      * @example
diff --git a/rhino_modules/jsdoc/src/filter.js b/rhino_modules/jsdoc/src/filter.js
index a2664305..573d9a54 100644
--- a/rhino_modules/jsdoc/src/filter.js
+++ b/rhino_modules/jsdoc/src/filter.js
@@ -14,10 +14,10 @@
  */
 exports.Filter = function(opts) {
     this.exclude = opts.exclude || null;
-    this.includePattern = opts.includePattern? 
+    this.includePattern = opts.includePattern?
                             typeof opts.includePattern === 'string'? new RegExp(opts.includePattern) : opts.includePattern
                             : null;
-    this.excludePattern = opts.excludePattern? 
+    this.excludePattern = opts.excludePattern?
                             typeof opts.excludePattern === 'string'? new RegExp(opts.excludePattern) : opts.excludePattern
                             : null;
 }
diff --git a/rhino_modules/jsdoc/tag/dictionary/definitions.js b/rhino_modules/jsdoc/tag/dictionary/definitions.js
index 239eebef..0e4b5e99 100644
--- a/rhino_modules/jsdoc/tag/dictionary/definitions.js
+++ b/rhino_modules/jsdoc/tag/dictionary/definitions.js
@@ -29,7 +29,7 @@ exports.defineTags = function(dictionary) {
                 doclet.access = tag.value.toLowerCase();
             }
             else {
-                delete doclet.access; 
+                delete doclet.access;
             }
         }
     });
@@ -292,7 +292,7 @@ exports.defineTags = function(dictionary) {
     
     dictionary.defineTag('instance', {
         onTagged: function(doclet, tag) {
-            setDocletScopeToTitle(doclet, tag);  
+            setDocletScopeToTitle(doclet, tag);
         }
     });
     
@@ -466,7 +466,7 @@ exports.defineTags = function(dictionary) {
     
     dictionary.defineTag('static', {
         onTagged: function(doclet, tag) {
-            setDocletScopeToTitle(doclet, tag);  
+            setDocletScopeToTitle(doclet, tag);
         }
     });
     
@@ -586,7 +586,7 @@ function setDocletDescriptionToValue(doclet, tag) {
 }
 
 function setNameToFile(doclet, tag) {
-    if (doclet.meta.filename) { 
+    if (doclet.meta.filename) {
         var name = 'file:';
         if (doclet.meta.path) { name += doclet.meta.path + java.lang.System.getProperty("file.separator"); }
         doclet.addTag( 'name', name + doclet.meta.filename );
diff --git a/rhino_modules/jsdoc/util/doop.js b/rhino_modules/jsdoc/util/doop.js
index 74ab75fe..a6b0732e 100644
--- a/rhino_modules/jsdoc/util/doop.js
+++ b/rhino_modules/jsdoc/util/doop.js
@@ -10,8 +10,8 @@ var doop = exports.doop = function(o) {
         clone = o instanceof Array ? [] : {};
         
         for (prop in o){
-            if ( hasOwnProp.call(o, prop) ) { 
-                clone[prop] = (o[prop] instanceof Object)? doop(o[prop]) : o[prop]; 
+            if ( hasOwnProp.call(o, prop) ) {
+                clone[prop] = (o[prop] instanceof Object)? doop(o[prop]) : o[prop];
             }
         }
         return clone;
diff --git a/templates/default/publish.js b/templates/default/publish.js
index d8a2e375..7081ee7c 100644
--- a/templates/default/publish.js
+++ b/templates/default/publish.js
@@ -93,8 +93,8 @@
                 types = types.map(function(t) {
                     return linkto(t, htmlsafe(t));
                 });
-            } 
-               
+            }
+            
             f.signature = (f.signature || '') + '<span class="type-signature">'+(types.length? ' :'+types.join('|') : '')+'</span>';
         }
         
@@ -327,7 +327,7 @@
 //                 }
 //                 seen[c.longname] = true;
 //             });
-//             
+//
 //             nav += '</ul>';
 //         }
         
diff --git a/templates/haruki/publish.js b/templates/haruki/publish.js
index fd8d5503..fcfc8e1a 100644
--- a/templates/haruki/publish.js
+++ b/templates/haruki/publish.js
@@ -40,7 +40,7 @@
     
     function graft(parentNode, childNodes, parentLongname, parentName) {
         childNodes
-        .filter(function (element) {            
+        .filter(function (element) {
             return (element.memberof === parentLongname);
         })
         .forEach(function (element, i) {