From 458ac1b01590506891b2ad2b300ead76aaa4e119 Mon Sep 17 00:00:00 2001 From: Jes Cok Date: Thu, 20 Feb 2025 15:10:06 +0000 Subject: [PATCH] context: replace newAfterFuncContext with a global declaration using blank identifier newAfterFuncContext has never been used, the only reason I can imagine for its existence is to guarantee that the implementation is correct. It is a small cleanup and make code more idiomatic. Change-Id: I61ee213a9284f3c3bda7f91196f3a1604babd0f6 GitHub-Last-Rev: c08bd69ef3f0dbc2f573f54095fea35dad2b1145 GitHub-Pull-Request: golang/go#71856 Reviewed-on: https://go-review.googlesource.com/c/go/+/651015 Auto-Submit: Dmitri Shuralyov LUCI-TryBot-Result: Go LUCI Reviewed-by: Damien Neil Reviewed-by: Dmitri Shuralyov --- src/context/afterfunc_test.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/context/afterfunc_test.go b/src/context/afterfunc_test.go index 7b75295eb4..c2ef1062d3 100644 --- a/src/context/afterfunc_test.go +++ b/src/context/afterfunc_test.go @@ -20,9 +20,7 @@ type afterFuncContext struct { err error } -func newAfterFuncContext() context.Context { - return &afterFuncContext{} -} +var _ context.Context = (*afterFuncContext)(nil) func (c *afterFuncContext) Deadline() (time.Time, bool) { return time.Time{}, false -- 2.51.0