fix: fix infinite loop on broken class body

closes #143
meriyah · Oct 28, 2020 · 22eb9f8 · 22eb9f8
1 parent ba98f66
commit 22eb9f8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/src/parser.ts b/src/parser.ts
@@ -8166,7 +8166,7 @@ export function parseClassBody(
   parser.flags = (parser.flags | Flags.HasConstructor) ^ Flags.HasConstructor;
 
   const body: (ESTree.MethodDefinition | ESTree.FieldDefinition)[] = [];
-  let decorators: ESTree.Decorator[] = [];
+  let decorators: ESTree.Decorator[];
 
   while (parser.token !== Token.RightBrace) {
     let length = 0;

diff --git a/src/token.ts b/src/token.ts
@@ -54,7 +54,7 @@ export const enum Token {
   RightBrace   = 15 | IsAutoSemicolon | IsClassField, // }
   RightParen   = 16, // )
   Semicolon    = 17 | IsAutoSemicolon | IsClassField, // ;
-  Comma        = 18 | IsClassField, // ,
+  Comma        = 18, // ,
   LeftBracket  = 19 | IsExpressionStart | IsPatternStart | IsMemberOrCallExpression, // [
   RightBracket = 20, // ]
   Colon        = 21, // :

diff --git a/test/parser/next/public-fields.ts b/test/parser/next/public-fields.ts
@@ -16,7 +16,9 @@ describe('Next - Public fields', () => {
     [
       'class C { #m = function() { return "bar"; }; Child = class extends C { access() { return super.#m; } method() { return super.#m(); } } }',
       Context.OptionsWebCompat | Context.OptionsNext
-    ]
+    ],
+    ['class A { a, b }', Context.None],
+    ['class A { a, b }', Context.OptionsNext]
   ]);
 
   for (const arg of [