From f7910128e790a4c86c88c4b5f7640cf7d71ac6e6 Mon Sep 17 00:00:00 2001 From: Keith Randall Date: Mon, 5 Aug 2013 15:08:37 -0700 Subject: [PATCH] reflect: Get rid of the test for the error message when 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 | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/src/pkg/reflect/all_test.go b/src/pkg/reflect/all_test.go index b905f93436c..93df4d13654 100644 --- a/src/pkg/reflect/all_test.go +++ b/src/pkg/reflect/all_test.go @@ -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 - } -}