]> Cypherpunks repositories - gostls13.git/commitdiff
flag: clarify that the main func at pkg.go.dev is part of a testing suite
authorZeke Lu <lvzecai@gmail.com>
Tue, 25 Oct 2022 23:18:45 +0000 (23:18 +0000)
committerGopher Robot <gobot@golang.org>
Wed, 26 Oct 2022 18:59:00 +0000 (18:59 +0000)
flag.Example() has this comment:

    ... one must execute, typically at the start of main (not init!):
      flag.Parse()
    We don't run it here because this is not a main function

This example function will be renamed to "main" at pkg.go.dev, which
makes the comment confusing.
See https://pkg.go.dev/flag#example-package.

This change modify the comment to clarify this situation.

Change-Id: I17357fdaaefe54791fff8fbbf6a33003af207f88
GitHub-Last-Rev: eeea8ce39cda3321d51c6cfe29fbcb2444fbf9cd
GitHub-Pull-Request: golang/go#56411
Reviewed-on: https://go-review.googlesource.com/c/go/+/445315
Run-TryBot: Ian Lance Taylor <iant@google.com>
Run-TryBot: Rob Pike <r@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Rob Pike <r@golang.org>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

src/flag/example_test.go

index 04a0d20ee4ed827f668bcd05df477a21f8371016..088447d43f6195cec3b72c45ac20801c4776ba43 100644 (file)
@@ -78,6 +78,8 @@ func Example() {
        // to enable the flag package to see the flags defined there, one must
        // execute, typically at the start of main (not init!):
        //      flag.Parse()
-       // We don't run it here because this is not a main function and
-       // the testing suite has already parsed the flags.
+       // We don't call it here because this code is a function called "Example"
+       // that is part of the testing suite for the package, which has already
+       // parsed the flags. When viewed at pkg.go.dev, however, the function is
+       // renamed to "main" and it could be run as a standalone example.
 }