linter

Author: ichiriac

src/ast.js

@@ -279,10 +279,10 @@ AST.prototype.prepare = function(kind, docs, parser) {
   };
   /**
    * Helper to change a node kind
-   * @param {String} newKind 
+   * @param {String} newKind
    */
   result.setKind = function(newKind) {
-    kind =  newKind;
+    kind = newKind;
   };
   return result;
 };

src/ast/array.js

@@ -25,7 +25,7 @@ const KIND = "array";
  *      "kind": "entry",
  *      "key": {"kind": "string", "value": "foo", "isDoubleQuote": false},
  *      "value": {"kind": "string", "value": "bar", "isDoubleQuote": false}
- *    }, 
+ *    },
  *    {"kind": "number", "value": "3"}
  *  ]
  * }

src/ast/node.js

@@ -25,7 +25,7 @@ const Node = function Node(kind, docs, location) {
 
 /**
  * Includes current token position of the parser
- * @param {*} parser 
+ * @param {*} parser
  */
 Node.prototype.includeToken = function(parser) {
   if (this.loc) {

src/parser.js

@@ -361,7 +361,7 @@ parser.prototype.expectEndOfStatement = function(node) {
   if (this.token === ";") {
     // include only real ';' statements
     // https://github.com/glayzzle/php-parser/issues/164
-    if (node && this.lexer.yytext === ';') {
+    if (node && this.lexer.yytext === ";") {
       node.includeToken(this);
     }
   } else if (this.token !== this.tok.T_INLINE_HTML && this.token !== this.EOF) {
@@ -442,7 +442,7 @@ parser.prototype.text = function() {
 /** consume the next token **/
 parser.prototype.next = function() {
   // prepare the back command
-  if (this.token !== ';' || this.lexer.yytext === ';') {
+  if (this.token !== ";" || this.lexer.yytext === ";") {
     // ignore '?>' from automated resolution
     // https://github.com/glayzzle/php-parser/issues/168
     this.prev = [

src/parser/array.js

@@ -54,7 +54,7 @@ module.exports = {
     const self = this;
     return this.read_list(
       function() {
-        return self.read_array_pair(shortForm)
+        return self.read_array_pair(shortForm);
       },
       ",",
       true
@@ -85,10 +85,7 @@ module.exports = {
       const expr = this.read_expr();
       if (this.token === this.tok.T_DOUBLE_ARROW) {
         if (this.next().token === "&") {
-          return entry(
-            expr,
-            this.next().read_variable(true, false, true)
-          );
+          return entry(expr, this.next().read_variable(true, false, true));
         } else {
           return entry(expr, this.read_expr());
         }

src/parser/expr.js

@@ -168,8 +168,8 @@ module.exports = {
         this.innerList = false;
         if (this.expect("=")) {
           return assign(
-            result(assignList, false), 
-            this.next().read_expr(), 
+            result(assignList, false),
+            this.next().read_expr(),
             "="
           );
         } else {
@@ -414,9 +414,9 @@ module.exports = {
     } else if (this.is("SCALAR")) {
       result = this.node();
       expr = this.read_scalar();
-      if (expr.kind === 'array' && expr.shortForm && this.token === '=') {
+      if (expr.kind === "array" && expr.shortForm && this.token === "=") {
         // list assign
-        let list = this.node('list')(expr.items, true);
+        let list = this.node("list")(expr.items, true);
         if (expr.loc) list.loc = expr.loc;
         let right = this.next().read_expr();
         return result("assign", list, right, "=");

src/parser/loops.js

@@ -122,10 +122,8 @@ module.exports = {
     }
 
     // grammatically correct but not supported by PHP
-    if (key && key.kind === 'list') {
-      this.raiseError(
-        "Fatal Error : Cannot use list as key element"
-      );
+    if (key && key.kind === "list") {
+      this.raiseError("Fatal Error : Cannot use list as key element");
     }
 
     if (this.expect(")")) this.next();
@@ -141,7 +139,7 @@ module.exports = {
   /**
    * Reads a foreach variable statement
    * ```ebnf
-   * foreach_variable = 
+   * foreach_variable =
    *    variable |
    *    '&' variable |
    *    T_LIST '(' assignment_list ')' |
@@ -157,7 +155,7 @@ module.exports = {
       this.next();
       if (!isShort && this.expect("(")) this.next();
       const assignList = this.read_array_pair_list(isShort);
-      if (this.expect(isShort ? "]": ")")) this.next();
+      if (this.expect(isShort ? "]" : ")")) this.next();
       return result(assignList, isShort);
     } else {
       return this.read_variable(false, false, false);