diff options
| author | Shenghou Ma <minux@golang.org> | 2019-10-24 21:29:30 -0400 |
|---|---|---|
| committer | Robert Griesemer <gri@golang.org> | 2019-10-28 23:34:13 +0000 |
| commit | 449b6abbacc464443a7faf166bf4db3df3e0f8da (patch) | |
| tree | f171c30ee234dd58962d21b47c773db1b2c85aff /test/typeswitch2b.go | |
| parent | bf670bd83c849b56426c63b49f28121210a627ca (diff) | |
| download | go-449b6abbacc464443a7faf166bf4db3df3e0f8da.tar.gz go-449b6abbacc464443a7faf166bf4db3df3e0f8da.zip | |
cmd/compile/internal/gc: reword "declared and not used" error message
"declared and not used" is technically correct, but might confuse
the user. Switching "and" to "but" will hopefully create the
contrast for the users: they did one thing (declaration), but
not the other --- actually using the variable.
This new message is still not ideal (specifically, declared is not
entirely precise here), but at least it matches the other parsers
and is one step in the right direction.
Change-Id: I725c7c663535f9ab9725c4b0bf35b4fa74b0eb20
Reviewed-on: https://go-review.googlesource.com/c/go/+/203282
Run-TryBot: Minux Ma <minux@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Robert Griesemer <gri@golang.org>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Diffstat (limited to 'test/typeswitch2b.go')
| -rw-r--r-- | test/typeswitch2b.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/typeswitch2b.go b/test/typeswitch2b.go index 135ae86cff..6da0d5fa6e 100644 --- a/test/typeswitch2b.go +++ b/test/typeswitch2b.go @@ -11,9 +11,9 @@ package main func notused(x interface{}) { // The first t is in a different scope than the 2nd t; it cannot - // be accessed (=> declared and not used error); but it is legal + // be accessed (=> declared but not used error); but it is legal // to declare it. - switch t := 0; t := x.(type) { // ERROR "declared and not used" + switch t := 0; t := x.(type) { // ERROR "declared but not used" case int: _ = t // this is using the t of "t := x.(type)" } |