]> Cypherpunks repositories - gostls13.git/commitdiff
go/format: skip go/ast's object resolution
authorDaniel Martí <mvdan@mvdan.cc>
Thu, 21 Apr 2022 13:06:20 +0000 (14:06 +0100)
committerDaniel Martí <mvdan@mvdan.cc>
Thu, 21 Apr 2022 21:07:18 +0000 (21:07 +0000)
Just like https://golang.org/cl/401454 removed the work from gofmt for a
nice ~5% speed-up in the default case, we can also use the option in the
equivalent go/format for programs which use it rather than gofmt,
as go/format makes no use of objects either.

No benchmark numbers as we already measured the ~5% speed-up with gofmt
in the other CL linked above.

See #46485.

Change-Id: Icbf98e6d46a616081314e2faa13f1dfade3bbaef
Reviewed-on: https://go-review.googlesource.com/c/go/+/401474
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Bryan Mills <bcmills@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
src/go/format/format.go

index fb87e84a4eb25febf8fe80b09014085c4d7b5e5e..3837cb461703ace14e18f27895dc35cf970b03d4 100644 (file)
@@ -38,7 +38,7 @@ const (
 
 var config = printer.Config{Mode: printerMode, Tabwidth: tabWidth}
 
-const parserMode = parser.ParseComments
+const parserMode = parser.ParseComments | parser.SkipObjectResolution
 
 // Node formats node in canonical gofmt style and writes the result to dst.
 //