mirror of
https://github.com/golang/go
synced 2024-11-26 22:51:23 -07:00
slices: explicitly discard results of some functions
This will otherwise trigger an "unusedresult" vet check. Fixes #64978 Change-Id: Ie19aded0f808d394f389452c3ff7f3edc1ed710d Reviewed-on: https://go-review.googlesource.com/c/go/+/554196 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Bryan Mills <bcmills@google.com>
This commit is contained in:
parent
c0693f648a
commit
8088b6db23
@ -557,7 +557,7 @@ func TestInsertPanics(t *testing.T) {
|
||||
{"with out-of-bounds index and = cap", a[:1:2], 2, b[:]},
|
||||
{"with out-of-bounds index and < cap", a[:1:3], 2, b[:]},
|
||||
} {
|
||||
if !panics(func() { Insert(test.s, test.i, test.v...) }) {
|
||||
if !panics(func() { _ = Insert(test.s, test.i, test.v...) }) {
|
||||
t.Errorf("Insert %s: got no panic, want panic", test.name)
|
||||
}
|
||||
}
|
||||
@ -684,7 +684,7 @@ func TestDeletePanics(t *testing.T) {
|
||||
{"s[i:j] is valid and j > len(s)", s, 0, 4},
|
||||
{"s[i:j] is valid and i == j > len(s)", s, 3, 3},
|
||||
} {
|
||||
if !panics(func() { Delete(test.s, test.i, test.j) }) {
|
||||
if !panics(func() { _ = Delete(test.s, test.i, test.j) }) {
|
||||
t.Errorf("Delete %s: got no panic, want panic", test.name)
|
||||
}
|
||||
}
|
||||
@ -906,10 +906,10 @@ func TestGrow(t *testing.T) {
|
||||
}
|
||||
|
||||
// Test number of allocations.
|
||||
if n := testing.AllocsPerRun(100, func() { Grow(s2, cap(s2)-len(s2)) }); n != 0 {
|
||||
if n := testing.AllocsPerRun(100, func() { _ = Grow(s2, cap(s2)-len(s2)) }); n != 0 {
|
||||
t.Errorf("Grow should not allocate when given sufficient capacity; allocated %v times", n)
|
||||
}
|
||||
if n := testing.AllocsPerRun(100, func() { Grow(s2, cap(s2)-len(s2)+1) }); n != 1 {
|
||||
if n := testing.AllocsPerRun(100, func() { _ = Grow(s2, cap(s2)-len(s2)+1) }); n != 1 {
|
||||
errorf := t.Errorf
|
||||
if race.Enabled || testenv.OptimizationOff() {
|
||||
errorf = t.Logf // this allocates multiple times in race detector mode
|
||||
@ -921,7 +921,7 @@ func TestGrow(t *testing.T) {
|
||||
var gotPanic bool
|
||||
func() {
|
||||
defer func() { gotPanic = recover() != nil }()
|
||||
Grow(s1, -1)
|
||||
_ = Grow(s1, -1)
|
||||
}()
|
||||
if !gotPanic {
|
||||
t.Errorf("Grow(-1) did not panic; expected a panic")
|
||||
@ -1037,7 +1037,7 @@ func TestReplacePanics(t *testing.T) {
|
||||
{"s[i:j] is valid and j > len(s)", s, nil, 0, 4},
|
||||
} {
|
||||
ss, vv := Clone(test.s), Clone(test.v)
|
||||
if !panics(func() { Replace(ss, test.i, test.j, vv...) }) {
|
||||
if !panics(func() { _ = Replace(ss, test.i, test.j, vv...) }) {
|
||||
t.Errorf("Replace %s: should have panicked", test.name)
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user