tree 79558c0146c25217c59d9e6fc8671e36740121a4
parent aa29eadba215f2fa2e1c7049da3f590db4e59683
author Heschi Kreinick <heschi@google.com> 1575315966 -0500
committer Heschi Kreinick <heschi@google.com> 1575318633 +0000

internal/imports: make ApplyFixes work despite syntax errors

ApplyFixes is only used by gopls, which cares a lot about files with syntax
errors, and not at all about files that aren't structurally valid. Use
the standard ParseFile function instead of goimports' weird one.

Adding a test is impractical because it seems to break type checking of
whatever package it's in.

Fixes golang/go#35915.

Change-Id: Iaf0e331978415428a422d942a1e0c5f6e66dc8a1
Reviewed-on: https://go-review.googlesource.com/c/tools/+/209579
Reviewed-by: Rebecca Stambler <rstambler@golang.org>