Walk SwitchCase nodes as separate nodes in the base walker for SwitchStatement

marijnh · marijnh · commit 570d513f8821 · 2024-09-08T12:22:15.000+02:00
Issue #555
diff --git a/acorn-walk/src/index.js b/acorn-walk/src/index.js
@@ -207,11 +207,7 @@ base.WithStatement = (node, st, c) => {
 }
 base.SwitchStatement = (node, st, c) => {
   c(node.discriminant, st, "Expression")
-  for (let cs of node.cases) {
-    if (cs.test) c(cs.test, st, "Expression")
-    for (let cons of cs.consequent)
-      c(cons, st, "Statement")
-  }
+  for (let cs of node.cases) c(cs, st)
 }
 base.SwitchCase = (node, st, c) => {
   if (node.test) c(node.test, st, "Expression")

Original file line number	Diff line number	Diff line change
`@@ -207,11 +207,7 @@ base.WithStatement = (node, st, c) => {`
`207`	`207`	`}`
`208`	`208`	`base.SwitchStatement = (node, st, c) => {`
`209`	`209`	`c(node.discriminant, st, "Expression")`
`210`		`- for (let cs of node.cases) {`
`211`		`- if (cs.test) c(cs.test, st, "Expression")`
`212`		`- for (let cons of cs.consequent)`
`213`		`- c(cons, st, "Statement")`
`214`		`- }`
	`210`	`+ for (let cs of node.cases) c(cs, st)`
`215`	`211`	`}`
`216`	`212`	`base.SwitchCase = (node, st, c) => {`
`217`	`213`	`if (node.test) c(node.test, st, "Expression")`