diff options
author | SparrowLii <liyuancylx@gmail.com> | 2020-08-24 14:43:32 +0800 |
---|---|---|
committer | Robert Griesemer <gri@golang.org> | 2020-08-25 17:04:40 +0000 |
commit | 41bc0a1713b9436e96c2d64211ad94e42cafd591 (patch) | |
tree | 25a4e76b64d355116a4d86e2d1910a0b6a294652 /src/math | |
parent | d3f6e2f30083749ce8ab0668e324a523b6ba9ead (diff) | |
download | go-41bc0a1713b9436e96c2d64211ad94e42cafd591.tar.gz go-41bc0a1713b9436e96c2d64211ad94e42cafd591.zip |
math/big: fix TestShiftOverlap for test -count arguments > 1
Don't overwrite incoming test data.
The change uses copy instead of assigning statement to avoid this.
Change-Id: Ib907101822d811de5c45145cb9d7961907e212c3
Reviewed-on: https://go-review.googlesource.com/c/go/+/250137
Run-TryBot: Emmanuel Odeke <emm.odeke@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Robert Griesemer <gri@golang.org>
Diffstat (limited to 'src/math')
-rw-r--r-- | src/math/big/arith_test.go | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/math/big/arith_test.go b/src/math/big/arith_test.go index 05136f1895..e2b982c89c 100644 --- a/src/math/big/arith_test.go +++ b/src/math/big/arith_test.go @@ -241,20 +241,20 @@ var argshrVU = []argVU{ } func testShiftFunc(t *testing.T, f func(z, x []Word, s uint) Word, a argVU) { - // save a.d for error message, or it will be overwritten. + // work on copy of a.d to preserve the original data. b := make([]Word, len(a.d)) copy(b, a.d) - z := a.d[a.zp : a.zp+a.l] - x := a.d[a.xp : a.xp+a.l] + z := b[a.zp : a.zp+a.l] + x := b[a.xp : a.xp+a.l] c := f(z, x, a.s) for i, zi := range z { if zi != a.r[i] { - t.Errorf("d := %v, %s(d[%d:%d], d[%d:%d], %d)\n\tgot z[%d] = %#x; want %#x", b, a.m, a.zp, a.zp+a.l, a.xp, a.xp+a.l, a.s, i, zi, a.r[i]) + t.Errorf("d := %v, %s(d[%d:%d], d[%d:%d], %d)\n\tgot z[%d] = %#x; want %#x", a.d, a.m, a.zp, a.zp+a.l, a.xp, a.xp+a.l, a.s, i, zi, a.r[i]) break } } if c != a.c { - t.Errorf("d := %v, %s(d[%d:%d], d[%d:%d], %d)\n\tgot c = %#x; want %#x", b, a.m, a.zp, a.zp+a.l, a.xp, a.xp+a.l, a.s, c, a.c) + t.Errorf("d := %v, %s(d[%d:%d], d[%d:%d], %d)\n\tgot c = %#x; want %#x", a.d, a.m, a.zp, a.zp+a.l, a.xp, a.xp+a.l, a.s, c, a.c) } } |