Update gulp-eslint and minor style fixes to adapt to new eslint version

Author: bjornharrtell

.eslintrc

@@ -48,7 +48,6 @@
         "no-caller": 2,
         "no-div-regex": 2,
         "no-else-return": 0,
-        "no-empty-label": 2,
         "no-eq-null": 2,
         "no-eval": 2,
         "no-extra-bind": 2,
@@ -156,12 +155,11 @@
         "semi": [2, "always"],
         "semi-spacing": 2,
         "sort-vars": 0,
-        "space-after-keywords": [2, "always"],
+        "keyword-spacing": 2,
         "space-before-blocks": [2, "always"],
         "space-in-brackets": 0, // TODO: enable?
         "space-in-parens": 0, // TODO: enable?
         "space-infix-ops": 2,
-        "space-return-throw-case": 2,
         "space-unary-ops": 2,
         "spaced-comment": [2, "always"],
         "wrap-regex": 0,

gulpfile.js

@@ -48,7 +48,7 @@ gulp.task('coverage', function(cb) {
 });
 
 gulp.task('lint', function() {
-    var pipeline = gulp.src(options.lintPaths)
+    return gulp.src(options.lintPaths)
         .pipe(eslint())
         .pipe(eslint.formatEach())
         .pipe(eslint.failOnError());

lib/jsdoc/opts/args.js

@@ -86,4 +86,5 @@ exports.get = function(name) {
     else if ( hasOwnProp.call(ourOptions, name) ) {
         return ourOptions[name];
     }
+    return undefined;
 };

lib/jsdoc/src/astnode.js

@@ -363,6 +363,7 @@ var getInfo = exports.getInfo = function(node) {
                     info.paramnames = getParamNames(definition.value);
                     return true;
                 }
+                return false;
             });
 
             break;

lib/jsdoc/src/handlers.js

@@ -299,6 +299,8 @@ function newSymbolDoclet(parser, docletSrc, e) {
 
     addDoclet(parser, newDoclet);
     e.doclet = newDoclet;
+
+    return true;
 }
 
 /**

lib/jsdoc/src/parser.js

@@ -319,6 +319,7 @@ Parser.prototype.getBasename = function(name) {
     if (name !== undefined) {
         return name.replace(/^([$a-z_][$a-z_0-9]*).*?$/i, '$1');
     }
+    return undefined;
 };
 
 // TODO: docs

lib/jsdoc/src/visitor.js

@@ -132,6 +132,7 @@ function findRestParam(params) {
             restParam = param;
             return true;
         }
+        return false;
     });
 
     return restParam;

lib/jsdoc/tag/dictionary/definitions.js

@@ -175,6 +175,7 @@ function parseBorrows(doclet, tag) {
         else if (m[1]) {
             return { target: m[1] };
         }
+        return {};
     } else {
         return {};
     }

lib/jsdoc/util/markdown.js

@@ -153,6 +153,7 @@ function getParseFunction(parserName, conf) {
     else {
         logger.error('Unrecognized Markdown parser "%s". Markdown support is disabled.',
             parserName);
+        return undefined;
     }
 }
 

package.json

@@ -21,14 +21,14 @@
     "js2xmlparser": "~1.0.0",
     "marked": "~0.3.5",
     "requizzle": "~0.2.1",
-    "strip-json-comments": "~1.0.4",
+    "strip-json-comments": "~2.0.1",
     "taffydb": "https://github.com/hegemonic/taffydb/tarball/7d100bcee0e997ee4977e273cdce60bd8933050e",
     "underscore": "~1.8.3",
     "wrench": "~1.5.9"
   },
   "devDependencies": {
     "gulp": "~3.9.1",
-    "gulp-eslint": "~1.1.1",
+    "gulp-eslint": "~2.0.0",
     "gulp-json-editor": "~2.2.1",
     "istanbul": "~0.4.2",
     "tv4": "https://github.com/hegemonic/tv4/tarball/own-properties"