Add dbscheme table isParenthesized

Author: owen-mc

go/downgrades/b4012cfa6fa7de1878c247155b7dfcd76df2604d/go.dbscheme

@@ -0,0 +1,3 @@
+description: Remove table recording the number of parentheses around expressions
+compatibility: full
+isParenthesized.rel: delete

go/downgrades/b4012cfa6fa7de1878c247155b7dfcd76df2604d/old.dbscheme

@@ -996,6 +996,12 @@ var ExprsTable = NewTable("exprs",
 	IntColumn("idx"),
 ).KeySet("parent", "idx")
 
+// IsParenthesizedTable is the table associating expressions with how many parentheses they originally had
+var IsParenthesizedTable = NewTable("isParenthesized",
+	EntityColumn(ExprType, "id").Unique(),
+	IntColumn("parentheses"),
+)
+
 // LiteralsTable is the table associating literal expression AST nodes with their values
 var LiteralsTable = NewTable("literals",
 	EntityColumn(ExprType, "expr").Unique(),

go/downgrades/b4012cfa6fa7de1878c247155b7dfcd76df2604d/upgrade.properties

@@ -1026,12 +1026,22 @@ func extractExpr(tw *trap.Writer, expr ast.Expr, parent trap.Label, idx int, ski
 	}
 
 	// Skip parenthesised expressions and extract their child directly in their place
-	if paren, ok := expr.(*ast.ParenExpr); ok {
-		extractExpr(tw, paren.X, parent, idx, skipExtractingValue)
-		return
+	nParens := 0
+	for {
+		paren, ok := expr.(*ast.ParenExpr)
+		if !ok {
+			break
+		}
+		nParens++
+		expr = paren.X
 	}
 
 	lbl := tw.Labeler.LocalID(expr)
+
+	if nParens > 0 {
+		dbscheme.IsParenthesizedTable.Emit(tw, lbl, nParens)
+	}
+
 	extractTypeOf(tw, expr, lbl)
 
 	var kind int

go/extractor/dbscheme/tables.go

@@ -162,6 +162,8 @@ doc_comments(unique int node: @documentable ref, int comment: @comment_group ref
 #keyset[parent, idx]
 exprs(unique int id: @expr, int kind: int ref, int parent: @exprparent ref, int idx: int ref);
 
+isParenthesized(unique int id: @expr ref, int parentheses: int ref);
+
 literals(unique int expr: @expr ref, string value: string ref, string raw: string ref);
 
 constvalues(unique int expr: @expr ref, string value: string ref, string exact: string ref);
@@ -561,4 +563,3 @@ case @error.kind of
 | 1 = @listerror
 | 2 = @parseerror
 | 3 = @typeerror;
-