]> Cypherpunks repositories - gostls13.git/commitdiff
os/exec: clarify Cmd.Wait documentation a bit more explicitly
authorBrad Fitzpatrick <bradfitz@golang.org>
Thu, 6 Jul 2017 17:56:21 +0000 (17:56 +0000)
committerBrad Fitzpatrick <bradfitz@golang.org>
Thu, 6 Jul 2017 19:29:20 +0000 (19:29 +0000)
It already implied that Cmd.Wait is more than os.Process.Wait, but say
so explicitly.

See https://github.com/golang/go/issues/18874#issuecomment-309921486

Updates #18874

Change-Id: Iaa46defd776ae0be817d9f4466a99ac78cfd672b
Reviewed-on: https://go-review.googlesource.com/47650
Reviewed-by: Russ Cox <rsc@golang.org>
src/os/exec/exec.go

index c3599681c316f2911773c90759a4620912c8bd6c..893d8ee99a8fd6a8c69bcca55a78a04c7fa90b7d 100644 (file)
@@ -416,8 +416,10 @@ func (e *ExitError) Error() string {
        return e.ProcessState.String()
 }
 
-// Wait waits for the command to exit.
-// It must have been started by Start.
+// Wait waits for the command to exit and waits for any copying to
+// stdin or copying from stdout or stderr to complete.
+//
+// The command must have been started by Start.
 //
 // The returned error is nil if the command runs, has no problems
 // copying stdin, stdout, and stderr, and exits with a zero exit