diff options
| author | Jes Cok <xigua67damn@gmail.com> | 2025-07-26 02:49:42 +0000 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2025-07-28 11:28:38 -0700 |
| commit | 64ba72474d9a637fc8152c3bbd8cdec3ffde2940 (patch) | |
| tree | 5091f3e71e4981ea99461432afa9b19acd46241b /src | |
| parent | e151db3e065eea8a13fd2bc83aafb6959edd6fca (diff) | |
| download | go-64ba72474d9a637fc8152c3bbd8cdec3ffde2940.tar.xz | |
errors: omit redundant nil check in type assertion for Join
When ok is true, err can't be nil.
Make it behave more like the Unwrap function.
Change-Id: Ieba5de57d60f5ff4d6a3468d703e6f72be02a97d
GitHub-Last-Rev: 6df9365a1d586bba64df57be11f9ca7b772cc773
GitHub-Pull-Request: golang/go#74764
Reviewed-on: https://go-review.googlesource.com/c/go/+/689920
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Jorropo <jorropo.pgm@gmail.com>
Reviewed-by: Keith Randall <khr@google.com>
Reviewed-by: Mark Freeman <mark@golang.org>
Reviewed-by: Jorropo <jorropo.pgm@gmail.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/errors/join.go | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/errors/join.go b/src/errors/join.go index dd50089c29..d0a1bc298e 100644 --- a/src/errors/join.go +++ b/src/errors/join.go @@ -28,12 +28,10 @@ func Join(errs ...error) error { } if n == 1 { for _, err := range errs { - if err != nil { - if _, ok := err.(interface { - Unwrap() []error - }); ok { - return err - } + if _, ok := err.(interface { + Unwrap() []error + }); ok { + return err } } } |