refactor(astutils): remove redundant nil check in Walk (#2660)

From the Go docs: "For a nil slice, the number of iterations is 0." [1] Therefore, an additional nil check for before the loop is unnecessary. [1]: https://go.dev/ref/spec#For_range Signed-off-by: Eng Zer Jun <[email protected]>
sqlc-dev · Aug 28, 2023 · ecf4980 · ecf4980
1 parent 8be0e2f
commit ecf4980
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/internal/sql/astutils/walk.go b/internal/sql/astutils/walk.go
@@ -2158,10 +2158,8 @@ func Walk(f Visitor, node ast.Node) {
 		}
 
 	case *ast.In:
-		if n.List != nil {
-			for _, l := range n.List {
-				Walk(f, l)
-			}
+		for _, l := range n.List {
+			Walk(f, l)
 		}
 		if n.Sel != nil {
 			Walk(f, n.Sel)