http: Support HTTP/1.0 Keep-Alive

http: Support HTTP/1.0 Keep-Alive

[rsc1, 2010-09-23 18:31:22]

LGTM but please fix the broken comment

http://codereview.appspot.com/2261042/diff/2001/src/pkg/http/server.go
File src/pkg/http/server.go (right):

http://codereview.appspot.com/2261042/diff/2001/src/pkg/http/server.go#newcode59
src/pkg/http/server.go:59: closeAfterReply bool // close connection after this
reply, unless
gofmt didn't like this comment break.  maybe reformat into
a multiline comment above the closeAfterReply bool line.

http://codereview.appspot.com/2261042/diff/2001/src/pkg/http/server.go#newcod...
src/pkg/http/server.go:365: // [*] Not strictly true: HTTP pipelining.  We could
let them all process
Thanks for adding the comment.
This is in the works.

[bradfitzgoog, 2010-09-26 16:09:10]

Ah whoops, didn't double-check the code after gofmt -w.  PTAL.

On Thu, Sep 23, 2010 at 11:31 AM, <rsc@google.com> wrote:

> LGTM but please fix the broken comment
>
>
>
> http://codereview.appspot.com/2261042/diff/2001/src/pkg/http/server.go
> File src/pkg/http/server.go (right):
>
>
>
http://codereview.appspot.com/2261042/diff/2001/src/pkg/http/server.go#newcode59
> src/pkg/http/server.go:59: closeAfterReply bool // close connection
> after this reply, unless
> gofmt didn't like this comment break.  maybe reformat into
> a multiline comment above the closeAfterReply bool line.
>
>
>
http://codereview.appspot.com/2261042/diff/2001/src/pkg/http/server.go#newcod...
> src/pkg/http/server.go:365: // [*] Not strictly true: HTTP pipelining.
> We could let them all process
> Thanks for adding the comment.
> This is in the works.
>
> http://codereview.appspot.com/2261042/
>

[rsc, 2010-09-28 01:55:05]

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

http: support HTTP/1.0 Keep-Alive

R=rsc, bradfitz1
CC=golang-dev
http://codereview.appspot.com/2261042

Committer: Russ Cox <rsc@golang.org>