code review 4550111: exec: missing docs, errors

exec: missing docs, errors

[bradfitz, 2011-06-02 14:52:09]

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

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

[rsc, 2011-06-02 15:04:05]

LGTM

http://codereview.appspot.com/4550111/diff/5001/src/pkg/exec/exec.go
File src/pkg/exec/exec.go (right):

http://codereview.appspot.com/4550111/diff/5001/src/pkg/exec/exec.go#newcode201
src/pkg/exec/exec.go:201: // Start starts the specified command but does not
wait for it
move to complete up here.
delete next sentence (implied by signature)

[bradfitz, 2011-06-02 16:34:11]

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

Please take another look.

[bradfitz, 2011-06-02 16:34:58]

Also made a double-Wait an error, to catch accidental Wait-after-Run.

On Thu, Jun 2, 2011 at 9:34 AM, <bradfitz@golang.org> wrote:

> Hello rsc@golang.org (cc: golang-dev@googlegroups.com),
>
> Please take another look.
>
>
>
> http://codereview.appspot.com/4550111/
>

[bradfitz, 2011-06-02 16:35:48]

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

Please take another look.

[rsc, 2011-06-02 16:56:07]

LGTM

[bradfitz, 2011-06-02 16:57:27]

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

exec: missing docs, errors

R=rsc
CC=golang-dev
http://codereview.appspot.com/4550111