fixprevent panics when encountering unexpected ast nodes in range function lists (#2449)

Author: andrewmbenton

internal/compiler/output_columns.go

@@ -503,18 +503,25 @@ func (c *Compiler) sourceTables(qc *QueryCatalog, node ast.Node) ([]*Table, erro
 		switch n := item.(type) {
 
 		case *ast.RangeFunction:
-			// If the function or table can't be found, don't error out.  There
-			// are many queries that depend on functions unknown to sqlc.
 			var funcCall *ast.FuncCall
 			switch f := n.Functions.Items[0].(type) {
 			case *ast.List:
-				funcCall = f.Items[0].(*ast.FuncCall)
+				switch fi := f.Items[0].(type) {
+				case *ast.FuncCall:
+					funcCall = fi
+				case *ast.SQLValueFunction:
+					continue // TODO handle this correctly
+				default:
+					continue
+			}
 			case *ast.FuncCall:
 				funcCall = f
 			default:
 				return nil, fmt.Errorf("sourceTables: unsupported function call type %T", n.Functions.Items[0])
 			}
 
+			// If the function or table can't be found, don't error out.  There
+			// are many queries that depend on functions unknown to sqlc.
 			fn, err := qc.GetFunc(funcCall.Func)
 			if err != nil {
 				continue

internal/endtoend/testdata/func_return/postgresql/pgx/v4/go/query.sql.go

@@ -7,4 +7,7 @@ WHERE first_name != '';
 /* name: GenerateSeries :many */
 SELECT ($1::inet) + i
 FROM generate_series(0, $2::int) AS i
-LIMIT 1;
+LIMIT 1;
+
+/* name: GetDate :one */
+SELECT * from CURRENT_DATE;

internal/endtoend/testdata/func_return/postgresql/pgx/v4/query.sql

@@ -7,4 +7,7 @@ WHERE first_name != '';
 /* name: GenerateSeries :many */
 SELECT ($1::inet) + i
 FROM generate_series(0, $2::int) AS i
-LIMIT 1;
+LIMIT 1;
+
+/* name: GetDate :one */
+SELECT * from CURRENT_DATE;

internal/endtoend/testdata/func_return/postgresql/pgx/v5/go/query.sql.go

@@ -7,4 +7,7 @@ WHERE first_name != '';
 /* name: GenerateSeries :many */
 SELECT ($1::inet) + i
 FROM generate_series(0, $2::int) AS i
-LIMIT 1;
+LIMIT 1;
+
+/* name: GetDate :one */
+SELECT * from CURRENT_DATE;