code review 9738052: testing: add test for issue 5599

testing: add test for issue 5599

Update issue 5599

[dave_cheney.net, 2013-05-31 12:24:52]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go

[r, 2013-05-31 12:31:14]

LGTM

[minux1, 2013-05-31 12:35:00]

seems not worth the trouble to split the actual fix and the test into two
CLs, esp. since the fix only requires insertion of one equal sign.

[dave_cheney.net, 2013-05-31 12:37:30]

Opps, I forgot the copyright notices. Adding them now.


On Fri, May 31, 2013 at 10:34 PM, minux <minux.ma@gmail.com> wrote:
> seems not worth the trouble to split the actual fix and the test into two
> CLs, esp. since the fix only requires insertion of one equal sign.

[dave_cheney.net, 2013-05-31 12:38:46]

Hi Minux,

I think the fix might be a little harder than expected, try it with
this CL. Anyway, I don't want to steal all of Keith's thunder.

On Fri, May 31, 2013 at 10:37 PM, Dave Cheney <dave@cheney.net> wrote:
> Opps, I forgot the copyright notices. Adding them now.
>
>
> On Fri, May 31, 2013 at 10:34 PM, minux <minux.ma@gmail.com> wrote:
>> seems not worth the trouble to split the actual fix and the test into two
>> CLs, esp. since the fix only requires insertion of one equal sign.

[dave_cheney.net, 2013-05-31 12:39:30]

Hello golang-dev@googlegroups.com, r@golang.org, minux.ma@gmail.com (cc:
golang-dev@googlegroups.com),

Please take another look.

[r, 2013-05-31 13:00:11]

LGTM

[dave_cheney.net, 2013-05-31 13:03:36]

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

testing: add test for issue 5599

Update issue 5599

R=golang-dev, r, minux.ma
CC=golang-dev
https://codereview.appspot.com/9738052