]> Cypherpunks repositories - gostls13.git/commitdiff
reflect: Get rid of the test for the error message when
authorKeith Randall <khr@golang.org>
Mon, 5 Aug 2013 22:08:37 +0000 (15:08 -0700)
committerKeith Randall <khr@golang.org>
Mon, 5 Aug 2013 22:08:37 +0000 (15:08 -0700)
you do reflect.call with too big an argument list.
Not worth the hassle.

Fixes #6023
Fixes #6033

R=golang-dev, bradfitz, dave
CC=golang-dev
https://golang.org/cl/12485043

src/pkg/reflect/all_test.go

index b905f93436cc848523a115123bdb37e497f1837b..93df4d13654af8da9c121ff4b4b91c279bcaff71 100644 (file)
@@ -3509,14 +3509,3 @@ func (x *exhaustive) Choose(max int) int {
 func (x *exhaustive) Maybe() bool {
        return x.Choose(2) == 1
 }
-
-func bigArgFunc(v [(1<<30)+64]byte) {
-}
-
-func TestBigArgs(t *testing.T) {
-       if !testing.Short() && ^uint(0)>>32 != 0 { // test on 64-bit only
-               v := new([(1<<30)+64]byte)
-               bigArgFunc(*v) // regular calls are ok
-               shouldPanic(func() {ValueOf(bigArgFunc).Call([]Value{ValueOf(*v)})}) // ... just not reflect calls
-       }
-}