From 1b3db48db7afc3fe17440af28cdeac67a0d048f1 Mon Sep 17 00:00:00 2001 From: Damien Neil Date: Mon, 22 Dec 2025 16:35:42 -0800 Subject: [PATCH] Revert "errors: optimize errors.Join for single unwrappable errors" This reverts CL 635115. Reason for revert: The new behavior does not match the function documentation. Fixes #76961 Change-Id: If2450aa4efba28c7a12887a5b306c231a836e740 Reviewed-on: https://go-review.googlesource.com/c/go/+/731981 Reviewed-by: Dmitri Shuralyov Reviewed-by: Dmitri Shuralyov Auto-Submit: Damien Neil LUCI-TryBot-Result: Go LUCI --- src/errors/join.go | 10 ---------- src/errors/join_test.go | 34 ---------------------------------- 2 files changed, 44 deletions(-) diff --git a/src/errors/join.go b/src/errors/join.go index 08a79867c6..730bf7043c 100644 --- a/src/errors/join.go +++ b/src/errors/join.go @@ -27,16 +27,6 @@ func Join(errs ...error) error { if n == 0 { return nil } - if n == 1 { - for _, err := range errs { - if _, ok := err.(interface { - Unwrap() []error - }); ok { - return err - } - } - } - e := &joinError{ errs: make([]error, 0, n), } diff --git a/src/errors/join_test.go b/src/errors/join_test.go index 439b372ca0..4828dc4d75 100644 --- a/src/errors/join_test.go +++ b/src/errors/join_test.go @@ -70,37 +70,3 @@ func TestJoinErrorMethod(t *testing.T) { } } } - -func BenchmarkJoin(b *testing.B) { - for _, bb := range []struct { - name string - errs []error - }{ - { - name: "no error", - }, - { - name: "single non-nil error", - errs: []error{errors.New("err")}, - }, - { - name: "multiple errors", - errs: []error{errors.New("err"), errors.New("newerr"), errors.New("newerr2")}, - }, - { - name: "unwrappable single error", - errs: []error{errors.Join(errors.New("err"))}, - }, - { - name: "nil first error", - errs: []error{nil, errors.New("newerr")}, - }, - } { - b.Run(bb.name, func(b *testing.B) { - b.ReportAllocs() - for i := 0; i < b.N; i++ { - _ = errors.Join(bb.errs...) - } - }) - } -} -- 2.52.0