mirror of https://go.googlesource.com/go
abf9b112fd
This CL re-introduces useLHS because we don't want to suppress correct "declared but not used" errors for variables that only appear on the LHS of an assignment (using Checker.use would mark them as used). This CL also adjusts a couple of places where types2 differed from go/types (and suppressed valid "declared and not used" errors). Now those errors are surfaced. Adjusted a handful of tests accordingly. Change-Id: Ia555139a05049887aeeec9e5221b1f41432c1a57 Reviewed-on: https://go-review.googlesource.com/c/go/+/478635 Reviewed-by: Robert Griesemer <gri@google.com> Auto-Submit: Robert Griesemer <gri@google.com> Reviewed-by: Robert Findley <rfindley@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Robert Griesemer <gri@google.com> |
||
|---|---|---|
| .. | ||
| embed1.dir | ||
| embed3.dir | ||
| private.dir | ||
| recursive1.dir | ||
| assertinline.go | ||
| bigdata.go | ||
| convert.go | ||
| convert1.go | ||
| convert2.go | ||
| embed.go | ||
| embed1.go | ||
| embed2.go | ||
| embed3.go | ||
| explicit.go | ||
| fail.go | ||
| fake.go | ||
| noeq.go | ||
| pointer.go | ||
| private.go | ||
| receiver.go | ||
| receiver1.go | ||
| recursive.go | ||
| recursive1.go | ||
| returntype.go | ||
| struct.go | ||