Merge pull request #367 from glayzzle/issue-358

fix: don't add unnecessary `noop`

Author: ichiriac

src/parser/array.js

@@ -64,10 +64,12 @@ module.exports = {
    */
   read_array_pair: function(shortForm) {
     if (
-      this.token === "," ||
       (!shortForm && this.token === ")") ||
       (shortForm && this.token === "]")
     ) {
+      return;
+    }
+    if (this.token === ",") {
       return this.node("noop")();
     }
     if (this.token === "&") {

src/parser/utils.js

@@ -40,7 +40,6 @@ module.exports = {
         break;
       }
       if (this.next().token == ")" && this.php73) {
-        result.push(this.node("noop")());
         break;
       }
     } while (this.token != this.EOF);
@@ -65,7 +64,10 @@ module.exports = {
 
     if (typeof item === "function") {
       do {
-        result.push(item.apply(this, []));
+        const itemResult = item.apply(this, []);
+        if (itemResult) {
+          result.push(itemResult);
+        }
         if (this.token != separator) {
           break;
         }