code review 91880044: net/http: remove a duplicated check

net/http: remove a duplicated check

The previous call to parseRange already checks whether
all the ranges start before the end of file.

[robryk, 2014-04-28 05:23:58]

Hello golang-codereviews@googlegroups.com,

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

[rh, 2014-04-28 12:17:39]

On 2014/04/28 05:23:58, robryk wrote:
> Hello mailto:golang-codereviews@googlegroups.com,
> 
> I'd like you to review this change to
> https://code.google.com/p/go/

LGTM

Nice catch.

[gobot, 2014-04-28 17:19:58]

R=bradfitz@golang.org (assigned by r@golang.org)

[robryk, 2014-06-19 21:52:45]

Gentle ping. Go 1.3 is out so this should be ok now.

[bradfitz, 2014-06-24 00:37:39]

LGTM

[bradfitz, 2014-06-24 00:38:20]

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

net/http: remove a duplicated check

The previous call to parseRange already checks whether
all the ranges start before the end of file.

LGTM=robert.hencke, bradfitz
R=golang-codereviews, robert.hencke, gobot, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/91880044

Committer: Brad Fitzpatrick <bradfitz@golang.org>

[gobot, 2014-06-24 01:24:02]

This CL appears to have broken the netbsd-amd64-bsiegert builder.
See http://build.golang.org/log/2d23e413055bb193c082e84785e8adce72376707