Re: [PATCH, go]: Do not panic in test/nilptr.go

2012-02-01 Thread Ian Lance Taylor
Uros Bizjak writes: > Attached is the patch I have committed. Thanks for taking care of that. > (BTW: Do you have any idea on what to do with excessive memory usage > in chan/select2.go? ) At this point I don't. It's sort of peculiar. Sending an int on a channel should not use any memory.

Re: [PATCH, go]: Do not panic in test/nilptr.go

2012-02-01 Thread Uros Bizjak
On Tue, Jan 31, 2012 at 8:36 PM, Ian Lance Taylor wrote: >> There is no need for a panic in test/nilptr.go if array doesn't get >> allocated in first 256 meg of memory. The compiler has nothing to do >> with this. > > This is true but this does not seem like the right patch. > I think I would pr

Re: [PATCH, go]: Do not panic in test/nilptr.go

2012-01-31 Thread Ian Lance Taylor
Uros Bizjak writes: > There is no need for a panic in test/nilptr.go if array doesn't get > allocated in first 256 meg of memory. The compiler has nothing to do > with this. This is true but this does not seem like the right patch. > Index: test/nilptr.go >

[PATCH, go]: Do not panic in test/nilptr.go

2012-01-30 Thread Uros Bizjak
Hello! There is no need for a panic in test/nilptr.go if array doesn't get allocated in first 256 meg of memory. The compiler has nothing to do with this. Index: test/nilptr.go === --- test/nilptr.go (revision 183732) +++ test/n