code review 5437154: test: make array smaller in nilptr test

test: make array smaller in nilptr test

Fixes issue 2314.

[rsc, 2011-12-07 16:44:50]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[r, 2011-12-07 16:49:03]

LGTM

[rsc, 2011-12-07 20:00:47]

*** Submitted as http://code.google.com/p/go/source/detail?r=579105168c89 ***

test: make array smaller in nilptr test

Fixes issue 2314.

R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/5437154

[dave_cheney.net, 2011-12-07 21:33:04]

Yay, that makes this test more reliable on 1gb OpenVZ VPS hosts. 

Sent from my iPhone

On 08/12/2011, at 3:44, rsc@golang.org wrote:

> Reviewers: golang-dev_googlegroups.com,
> 
> Message:
> Hello golang-dev@googlegroups.com,
> 
> I'd like you to review this change to
> https://go.googlecode.com/hg/
> 
> 
> Description:
> test: make array smaller in nilptr test
> 
> Fixes issue 2314.
> 
> Please review this at http://codereview.appspot.com/5437154/
> 
> Affected files:
>  M test/nilptr.go
> 
> 
> Index: test/nilptr.go
> ===================================================================
> --- a/test/nilptr.go
> +++ b/test/nilptr.go
> @@ -13,7 +13,7 @@
> // cause a memory access fault. This test checks
> // that Go is doing the correct explicit checks to catch
> // these nil pointer accesses, not just relying on the hardware.
> -var dummy [512 << 20]byte // give us a big address space
> +var dummy [256 << 20]byte // give us a big address space
> 
> func main() {
>    // the test only tests what we intend to test
> 
>