]> Cypherpunks repositories - gostls13.git/commitdiff
os/signal: improve documentation for the Notify function
authorRadek Sohlich <sohlich@gmail.com>
Tue, 31 Oct 2017 16:59:29 +0000 (17:59 +0100)
committerIan Lance Taylor <iant@golang.org>
Wed, 1 Nov 2017 18:38:23 +0000 (18:38 +0000)
It is easy to miss the documentation information that no arguments
in the Notify function means that the Notify will catch all possible signals.
So the example was added with explicit comment above the Notify usage.

Fixes #22257

Change-Id: Ia6a16dd4a419f7c77d89020ca5db85979b5b474e
Reviewed-on: https://go-review.googlesource.com/74730
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
src/os/signal/example_test.go

index 5dfbe5f26bd37e0a1058eaeeaacb4685c69a7f0a..ecefc757b4ce9797027f72e8498a06ade2b3dea3 100644 (file)
@@ -21,3 +21,18 @@ func ExampleNotify() {
        s := <-c
        fmt.Println("Got signal:", s)
 }
+
+func ExampleNotify_allSignals() {
+       // Set up channel on which to send signal notifications.
+       // We must use a buffered channel or risk missing the signal
+       // if we're not ready to receive when the signal is sent.
+       c := make(chan os.Signal, 1)
+
+       // Passing no signals to Notify means that
+       // all signals will be sent to the channel.
+       signal.Notify(c)
+
+       // Block until any signal is received.
+       s := <-c
+       fmt.Println("Got signal:", s)
+}