code review 3071041: flag: handle multiple calls to flag.Parse (issue3071041)

[r...@golang.org]

Reviewers: golang-dev_googlegroups.com,

Message:
Hello golan...@googlegroups.com,

I'd like you to review this change.

Description:
flag: handle multiple calls to flag.Parse

Please review this at http://codereview.appspot.com/3071041/

Affected files:
M src/pkg/flag/flag.go

[r...@google.com]

R=r

This change keeps flag.Parse from crashing if called multiple times.
It also lets one do things like

flag.Parse()
switch flag.Arg(0) {
case "cmd1":
... add more flags
case "cmd2":
... add more flags
}
os.Args = flag.Args()
flag.Parse()

to implement the common subcommand
command line parsing found in tools like
Mercurial. I'm not sure if that's a good thing,
but not crashing certainly is.

http://codereview.appspot.com/3071041/

[r...@golang.org]

looks good but wants a test if feasible and maybe a sentence of new
package documentation

http://codereview.appspot.com/3071041/

[r...@golang.org]

Hello r (cc: golan...@googlegroups.com),

Please take another look.

http://codereview.appspot.com/3071041/

[r...@golang.org]

LGTM

http://codereview.appspot.com/3071041/

[r...@golang.org]

*** Submitted as cc69ceb37529 ***

flag: handle multiple calls to flag.Parse

R=r
CC=golang-dev
http://codereview.appspot.com/3071041

http://codereview.appspot.com/3071041/