]> Cypherpunks repositories - gostls13.git/commit
os: use AddCleanup, not SetFinalizer, for Process
authorIan Lance Taylor <iant@golang.org>
Mon, 23 Dec 2024 05:23:55 +0000 (21:23 -0800)
committerGopher Robot <gobot@golang.org>
Sat, 8 Feb 2025 00:21:02 +0000 (16:21 -0800)
commitbdbc5ca1bd0547d91441c4785854c82dbd852443
treeaf5ef4c789658b3d478fb97224d101346700af4b
parent8028731648eb8c5f7693a69f748d990a3228a6e0
os: use AddCleanup, not SetFinalizer, for Process

There is no reason to use a cleanup/finalizer for a Process that
doesn't use a handle, because Release doesn't change anything visible
about the process.

For #70907

Change-Id: I3b92809175523ceee2e07d601cc2a8e8b86321e0
Reviewed-on: https://go-review.googlesource.com/c/go/+/638579
Reviewed-by: Carlos Amedee <carlos@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
src/os/exec.go
src/os/exec_plan9.go
src/os/exec_unix.go
src/os/exec_windows.go