Stowage/go
2
0
Fork 0
mirror of https://github.com/golang/go.git synced 2025-12-08 06:10:04 +00:00
Code Issues Projects Releases Packages Wiki Activity

reflect: Get rid of the test for the error message when

Browse source 
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
This commit is contained in:
Keith Randall 2013-08-05 15:08:37 -07:00
parent 598c78967f
commit f7910128e7
1 changed files with 0 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

11
src/pkg/reflect/all_test.go
View file

@ -3509,14 +3509,3 @@ func (x *exhaustive) Choose(max int) int {
func (x *exhaustive) Maybe() bool { func (x *exhaustive) Maybe() bool {
return x.Choose(2) == 1 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
}
}