code review 261041: fix bug in complex printing: imaginary didn't have same...

fix bug in complex printing: imaginary didn't have same format as real.
add tests.

[r, 2010-03-06 21:00:18]

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

I'd like you to review this change.

[r, 2010-03-06 21:05:44]

fix reviewer

[ken3, 2010-03-06 21:10:02]

i didnt include any print tests because
8g cant execute it yet and you will
end up with different golden files.

On Sat, Mar 6, 2010 at 1:00 PM,  <r@golang.org> wrote:
> Reviewers: rsc, yuri.vashchenko_gmail.com,
>
> Message:
> Hello rsc, ken (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> fix bug in complex printing: imaginary didn't have same format as real.
> add tests.
>
> Please review this at http://codereview.appspot.com/261041/show
>
> Affected files:
>  M src/pkg/fmt/fmt_test.go
>  M src/pkg/fmt/format.go
>  M src/pkg/fmt/print.go
>
>
>

[ken3, 2010-03-06 21:12:45]

LGTM

[r2, 2010-03-06 21:36:59]

OK. I'll comment out the tests for now, but the fixes are still valuable.

-rob

On Mar 6, 2010, at 1:09 PM, Ken Thompson wrote:

> i didnt include any print tests because
> 8g cant execute it yet and you will
> end up with different golden files.
> 
> On Sat, Mar 6, 2010 at 1:00 PM,  <r@golang.org> wrote:
>> Reviewers: rsc, yuri.vashchenko_gmail.com,
>> 
>> Message:
>> Hello rsc, ken (cc: golang-dev@googlegroups.com),
>> 
>> I'd like you to review this change.
>> 
>> 
>> Description:
>> fix bug in complex printing: imaginary didn't have same format as real.
>> add tests.
>> 
>> Please review this at http://codereview.appspot.com/261041/show
>> 
>> Affected files:
>>  M src/pkg/fmt/fmt_test.go
>>  M src/pkg/fmt/format.go
>>  M src/pkg/fmt/print.go
>> 
>> 
>> 

[r, 2010-03-06 21:38:18]

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

fix bug in complex printing: imaginary didn't have same format as real.
add tests.

R=rsc, ken2, ken3
CC=golang-dev
http://codereview.appspot.com/261041