From 644d479a27c0eccfc0b37e1a560ca09e47b5a972 Mon Sep 17 00:00:00 2001 From: Vitaly Zdanevich Date: Mon, 8 Mar 2021 05:54:42 +0300 Subject: [PATCH] errors: do not call Elem() twice --- src/errors/wrap.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/errors/wrap.go b/src/errors/wrap.go index 7928fe673e..4eb4f9ae37 100644 --- a/src/errors/wrap.go +++ b/src/errors/wrap.go @@ -83,10 +83,10 @@ func As(err error, target interface{}) bool { if typ.Kind() != reflectlite.Ptr || val.IsNil() { panic("errors: target must be a non-nil pointer") } - if e := typ.Elem(); e.Kind() != reflectlite.Interface && !e.Implements(errorType) { + targetType := typ.Elem() + if targetType.Kind() != reflectlite.Interface && !targetType.Implements(errorType) { panic("errors: *target must be interface or implement error") } - targetType := typ.Elem() for err != nil { if reflectlite.TypeOf(err).AssignableTo(targetType) { val.Elem().Set(reflectlite.ValueOf(err))