Skip to content

Commit e4b12eb

Browse files
cagedmantisgopherbot
authored andcommitted
io: use runtime.AddCleanup instead of runtime.SetFinalizer
Replace the usage of runtime.SetFinalizer with runtime.AddCleanup. Updates #70907 Change-Id: Id604ca44ea67dcf8f87797e27347c6f4e9ad0b86 Reviewed-on: https://go-review.googlesource.com/c/go/+/638556 Reviewed-by: Michael Pratt <[email protected]> Auto-Submit: Carlos Amedee <[email protected]> TryBot-Bypass: Carlos Amedee <[email protected]>
1 parent f7becfc commit e4b12eb

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

src/io/multi_test.go

+1-3
Original file line numberDiff line numberDiff line change
@@ -332,9 +332,7 @@ func TestMultiReaderFreesExhaustedReaders(t *testing.T) {
332332
buf1 := bytes.NewReader([]byte("foo"))
333333
buf2 := bytes.NewReader([]byte("bar"))
334334
mr = MultiReader(buf1, buf2)
335-
runtime.SetFinalizer(buf1, func(*bytes.Reader) {
336-
close(closed)
337-
})
335+
runtime.AddCleanup(buf1, func(ch chan struct{}) { close(ch) }, closed)
338336
}()
339337

340338
buf := make([]byte, 4)

0 commit comments

Comments
 (0)