opt(export): remember visited to avoid redundant computation (#46)

Hoblovski · web-flow · commit 92d8de90b64e · 2025-07-14T23:36:57.000+08:00
diff --git a/lang/collect/export.go b/lang/collect/export.go
@@ -75,8 +75,8 @@ func (c *Collector) Export(ctx context.Context) (*uniast.Repository, error) {
 	c.filterLocalSymbols()
 
 	// export symbols
+	visited := make(map[*lsp.DocumentSymbol]*uniast.Identity)
 	for _, symbol := range c.syms {
-		visited := make(map[*lsp.DocumentSymbol]*uniast.Identity)
 		_, _ = c.exportSymbol(&repo, symbol, "", visited)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -75,8 +75,8 @@ func (c Collector) Export(ctx context.Context) (uniast.Repository, error) {`
`75`	`75`	`c.filterLocalSymbols()`
`76`	`76`
`77`	`77`	`// export symbols`
	`78`	`+ visited := make(map[lsp.DocumentSymbol]uniast.Identity)`
`78`	`79`	`for _, symbol := range c.syms {`
`79`		`- visited := make(map[lsp.DocumentSymbol]uniast.Identity)`
`80`	`80`	`_, _ = c.exportSymbol(&repo, symbol, "", visited)`
`81`	`81`	`}`
`82`	`82`