commit | 4922717de774b11258a5f3b352bdbe3fd69fb2d1 | [log] [tgz] |
---|---|---|
author | Pontus Leitzler <leitzler@gmail.com> | Tue Jan 19 20:40:15 2021 +0100 |
committer | Rebecca Stambler <rstambler@golang.org> | Mon Jan 25 18:11:30 2021 +0000 |
tree | 8a3cd1ed56f70b1f88027fa9e7777c9c55eea531 | |
parent | 917f61dfb7ddaf4154035d0e21f74d4ab6982007 [diff] |
go/analysis/passes/fieldalignment: delete doc style comments in fix CL 278872 fixed the fact that field comments messed up the suggested edits by removing comments entirely. This change also remove documentation style comments since they also produce broken suggestions in the same way. Change-Id: I18d9c42fbf1453dcd5019b2a4aa3f4be4a58f7c7 Reviewed-on: https://go-review.googlesource.com/c/tools/+/284732 Trust: Rebecca Stambler <rstambler@golang.org> Trust: Pontus Leitzler <leitzler@gmail.com> Run-TryBot: Rebecca Stambler <rstambler@golang.org> gopls-CI: kokoro <noreply+kokoro@google.com> TryBot-Result: Go Bot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
diff --git a/go/analysis/passes/fieldalignment/fieldalignment.go b/go/analysis/passes/fieldalignment/fieldalignment.go index ca1bc53..ca7ceb2 100644 --- a/go/analysis/passes/fieldalignment/fieldalignment.go +++ b/go/analysis/passes/fieldalignment/fieldalignment.go
@@ -79,6 +79,7 @@ // TODO: Preserve comment, for now get rid of them. // See https://github.com/golang/go/issues/20744 f.Comment = nil + f.Doc = nil if len(f.Names) <= 1 { flat = append(flat, f) continue
diff --git a/go/analysis/passes/fieldalignment/testdata/src/a/a.go b/go/analysis/passes/fieldalignment/testdata/src/a/a.go index b47ee19..463b4cb 100644 --- a/go/analysis/passes/fieldalignment/testdata/src/a/a.go +++ b/go/analysis/passes/fieldalignment/testdata/src/a/a.go
@@ -35,3 +35,12 @@ y int32 z byte } + +type WithComments struct { // want "struct of size 8 could be 4" + // doc style comment + a uint32 // field a comment + b [0]byte // field b comment + // other doc style comment + + // and a last comment +}
diff --git a/go/analysis/passes/fieldalignment/testdata/src/a/a.go.golden b/go/analysis/passes/fieldalignment/testdata/src/a/a.go.golden index 34fc21b..c1c75e2 100644 --- a/go/analysis/passes/fieldalignment/testdata/src/a/a.go.golden +++ b/go/analysis/passes/fieldalignment/testdata/src/a/a.go.golden
@@ -35,3 +35,8 @@ x byte z byte } + +type WithComments struct { + b [0]byte + a uint32 +}