]> Cypherpunks repositories - gostls13.git/commitdiff
syscall: remove a wrong comment in Clearenv
authorKir Kolyshkin <kolyshkin@gmail.com>
Tue, 10 Dec 2024 20:10:13 +0000 (12:10 -0800)
committerGopher Robot <gobot@golang.org>
Wed, 11 Dec 2024 15:58:51 +0000 (07:58 -0800)
The comment being removed was added by commit ff3173849e
(which predates Gerrit and Rietveld, so no CL link), and
at the time it made sense.

Since CL 148370043 (and up to the current implementation of Clearenv)
the env map, which is populated by copyenv, is actually used, so the
comment is no longer valid.

It is also misleading, so it's best to remove it.

Change-Id: I8bd2e8bca6262759538e5bcbd396f0c71cca6a4c
Reviewed-on: https://go-review.googlesource.com/c/go/+/635078
Reviewed-by: Carlos Amedee <carlos@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>

src/syscall/env_unix.go

index 8e87e018e82de37d90c064df3bb6d8d7d3c57f8b..1144ed1416c0ee5e36b36f59ca870931efee5ee9 100644 (file)
@@ -124,7 +124,7 @@ func Setenv(key, value string) error {
 }
 
 func Clearenv() {
-       envOnce.Do(copyenv) // prevent copyenv in Getenv/Setenv
+       envOnce.Do(copyenv)
 
        envLock.Lock()
        defer envLock.Unlock()