code review 7390047: runtime: better error from TestGcSys when gc is disabled

runtime: better error from TestGcSys when gc is disabled

[rsc, 2013-02-21 18:22:32]

Hello golang-dev@googlegroups.com,

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

[rsc, 2013-02-21 18:22:37]

Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),

Please take another look.

[bradfitz, 2013-02-21 18:23:40]

why not t.Skip?

On Thu, Feb 21, 2013 at 10:22 AM, <rsc@golang.org> wrote:

> Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),
>
> Please take another look.
>
>
>
https://codereview.appspot.**com/7390047/<https://codereview.appspot.com/7390...
>
>
> --
>
> ---You received this message because you are subscribed to the Google
> Groups "golang-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to
golang-dev+unsubscribe@**googlegroups.com<golang-dev%2Bunsubscribe@googlegrou...
> .
> For more options, visit
https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/o...
> .
>
>
>

[rsc, 2013-02-21 18:24:45]

running tests with GOGC=off is not a mode i want to actively support.
i just don't want to puzzle over it the next time i do it. :-)

russ

[bradfitz, 2013-02-21 18:30:07]

LGTM

On Thu, Feb 21, 2013 at 10:24 AM, Russ Cox <rsc@golang.org> wrote:

> running tests with GOGC=off is not a mode i want to actively support.
> i just don't want to puzzle over it the next time i do it. :-)
>
> russ
>
>

[rsc, 2013-02-21 18:30:36]

*** Submitted as https://code.google.com/p/go/source/detail?r=ddf29de95e90 ***

runtime: better error from TestGcSys when gc is disabled

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/7390047