requireParamTypes: Change if to early return format

dtracers · Alexej Yaroshevich · commit 85fba47e7fa6 · 2015-06-24T17:45:18.000+03:00
- The if statement in the require param type rule uses the quick bail format Closes gh-112
diff --git a/lib/rules/validate-jsdoc/require-param-types.js b/lib/rules/validate-jsdoc/require-param-types.js
@@ -14,7 +14,8 @@ module.exports.options = {
  */
 function validateParamTag(node, tag, err) {
     // checking existance
-    if (!tag.type) {
-        return err('Missing param type', (tag.name ? tag.name.loc : null) || tag.loc);
+    if (tag.type) {
+        return;
     }
+    return err('Missing param type', (tag.name ? tag.name.loc : null) || tag.loc);
 }

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,8 @@ module.exports.options = {`
`14`	`14`	`*/`
`15`	`15`	`function validateParamTag(node, tag, err) {`
`16`	`16`	`// checking existance`
`17`		`- if (!tag.type) {`
`18`		`- return err('Missing param type', (tag.name ? tag.name.loc : null) \|\| tag.loc);`
	`17`	`+ if (tag.type) {`
	`18`	`+ return;`
`19`	`19`	`}`
	`20`	`+ return err('Missing param type', (tag.name ? tag.name.loc : null) \|\| tag.loc);`
`20`	`21`	`}`