tree 74a929f3f77a764b3adc74f3b43ff50c8077efa3
parent 330b9f13843665b0e77c9a64b2873b8d04a6df67
author Heschi Kreinick <heschi@google.com> 1575593354 -0500
committer Heschi Kreinick <heschi@google.com> 1575663686 +0000

internal/lsp/source: optimize computeFixEdits

In the common case that a file has imports, we're going to diff just the
import block. That means that ApplyFixes doesn't need to produce the
whole formatted file, which is a huge speedup. We will do more work twice
for files with no imports, but those are presumably pretty short.

Hat tip to Muir for pointing towards this in
https://go-review.googlesource.com/c/tools/+/209579/2/internal/imports/imports.go#87
even if I didn't catch it.

Updates golang/go#36001.

Change-Id: Ibbeb4d88c6505eac26a36994de514813606c8c79
Reviewed-on: https://go-review.googlesource.com/c/tools/+/210200
Run-TryBot: Heschi Kreinick <heschi@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Rebecca Stambler <rstambler@golang.org>