fix(es/parser): Do not parse empty stmt after using decl (#9798)

swc-project · Dec 15, 2024 · c2696db · c2696db
1 parent 46cbc8a
commit c2696db
Show file tree

Hide file tree

Showing 54 changed files with 121 additions and 904 deletions.
diff --git a/.changeset/lucky-hotels-grow.md b/.changeset/lucky-hotels-grow.md
@@ -0,0 +1,6 @@
+---
+swc_core: patch
+swc_ecma_parser: patch
+---
+
+fix(es/parser): Do not parse empty stmt after using decl
diff --git a/crates/swc_ecma_parser/src/parser/stmt.rs b/crates/swc_ecma_parser/src/parser/stmt.rs
@@ -839,6 +839,8 @@ impl<'a, I: Tokens> Parser<I> {
             }
         }
 
+        eat!(self, ';');
+
         Ok(Some(Box::new(UsingDecl {
             span: span!(self, start),
             is_await,

diff --git a/...ecma_parser/tests/js/explicit-resource-management/valid-using-binding-basic/input.js.json b/...ecma_parser/tests/js/explicit-resource-management/valid-using-binding-basic/input.js.json
@@ -17,7 +17,7 @@
           "type": "UsingDeclaration",
           "span": {
             "start": 5,
-            "end": 30
+            "end": 31
           },
           "isAwait": false,
           "decls": [
@@ -61,13 +61,6 @@
               "definite": false
             }
           ]
-        },
-        {
-          "type": "EmptyStatement",
-          "span": {
-            "start": 30,
-            "end": 31
-          }
         }
       ]
     }

diff --git a/...ma_parser/tests/js/explicit-resource-management/valid-using-binding-escaped/input.js.json b/...ma_parser/tests/js/explicit-resource-management/valid-using-binding-escaped/input.js.json
@@ -17,7 +17,7 @@
           "type": "UsingDeclaration",
           "span": {
             "start": 3,
-            "end": 20
+            "end": 21
           },
           "isAwait": false,
           "decls": [
@@ -51,13 +51,6 @@
               "definite": false
             }
           ]
-        },
-        {
-          "type": "EmptyStatement",
-          "span": {
-            "start": 20,
-            "end": 21
-          }
         }
       ]
     }

diff --git a/...ma_parser/tests/js/explicit-resource-management/valid-using-binding-non-bmp/input.js.json b/...ma_parser/tests/js/explicit-resource-management/valid-using-binding-non-bmp/input.js.json
@@ -17,7 +17,7 @@
           "type": "UsingDeclaration",
           "span": {
             "start": 3,
-            "end": 21
+            "end": 22
           },
           "isAwait": false,
           "decls": [
@@ -61,13 +61,6 @@
               "definite": false
             }
           ]
-        },
-        {
-          "type": "EmptyStatement",
-          "span": {
-            "start": 21,
-            "end": 22
-          }
         }
       ]
     }

diff --git a/...ecma_parser/tests/js/explicit-resource-management/valid-using-binding-using/input.js.json b/...ecma_parser/tests/js/explicit-resource-management/valid-using-binding-using/input.js.json
@@ -17,7 +17,7 @@
           "type": "UsingDeclaration",
           "span": {
             "start": 5,
-            "end": 21
+            "end": 22
           },
           "isAwait": false,
           "decls": [
@@ -52,13 +52,6 @@
             }
           ]
         },
-        {
-          "type": "EmptyStatement",
-          "span": {
-            "start": 21,
-            "end": 22
-          }
-        },
         {
           "type": "ForOfStatement",
           "span": {