code review 5488085: undo CL 5477092 / c3c6e72d7cc5

undo CL 5477092 / c3c6e72d7cc5

The obvious fix is breaking the build in non-obvious ways.
Reverting while waiting for the correct fix, if any is needed.

««« original CL description
net/http: fix bug in error checking

Thanks to josef86@gmail.com for pointing this out.

R=golang-dev, iant
CC=golang-dev
http://codereview.appspot.com/5477092
»»»

[gri, 2011-12-14 18:44:32]

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

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

[gri, 2011-12-14 18:44:40]

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

undo CL 5477092 / c3c6e72d7cc5

The obvious fix is breaking the build in non-obvious ways.
Reverting while waiting for the correct fix, if any is needed.

««« original CL description
net/http: fix bug in error checking

Thanks to josef86@gmail.com for pointing this out.

R=golang-dev, iant
CC=golang-dev
http://codereview.appspot.com/5477092
»»»

R=iant
CC=golang-dev
http://codereview.appspot.com/5488085

[iant2, 2011-12-14 19:14:22]

LGTM

gri@golang.org writes:

> Reviewers: iant,
>
> Message:
> Hello iant@golang.org (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change to
> https://go.googlecode.com/hg/
>
>
> Description:
> undo CL 5477092 / c3c6e72d7cc5
>
> The obvious fix is breaking the build in non-obvious ways.
> Reverting while waiting for the correct fix, if any is needed.
>
> ««« original CL description
> net/http: fix bug in error checking
>
> Thanks to josef86@gmail.com for pointing this out.
>
> R=golang-dev, iant
> CC=golang-dev
> http://codereview.appspot.com/5477092
> »»»
>
> Please review this at http://codereview.appspot.com/5488085/
>
> Affected files:
>   M src/pkg/net/http/transport.go
>
>
> Index: src/pkg/net/http/transport.go
> ===================================================================
> --- a/src/pkg/net/http/transport.go
> +++ b/src/pkg/net/http/transport.go
> @@ -544,7 +544,7 @@
>  				resp.Header.Del("Content-Length")
>  				resp.ContentLength = -1
>  				gzReader, zerr := gzip.NewReader(resp.Body)
> -				if zerr != nil {
> +				if err != nil {
>  					pc.close()
>  					err = zerr
>  				} else {