From 7e1260f62b7447c36d9f3ff95365d761592323c4 Mon Sep 17 00:00:00 2001 From: sivchari Date: Sat, 27 Nov 2021 23:02:16 +0000 Subject: [PATCH] testing: simplify fuzzResult.String to avoid unnecessarily using fmt.Sprintf Change-Id: I16b6bfb6b0f02672c894b20845aa14d8dd1979b4 GitHub-Last-Rev: 75ab90123a8a5ad42e96795b756d3a9e898aaa06 GitHub-Pull-Request: golang/go#49819 Reviewed-on: https://go-review.googlesource.com/c/go/+/367314 Reviewed-by: Emmanuel Odeke Reviewed-by: Ian Lance Taylor Run-TryBot: Emmanuel Odeke TryBot-Result: Go Bot --- src/testing/fuzz.go | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/testing/fuzz.go b/src/testing/fuzz.go index 24a0080730..19ff39947b 100644 --- a/src/testing/fuzz.go +++ b/src/testing/fuzz.go @@ -423,12 +423,10 @@ type fuzzResult struct { } func (r fuzzResult) String() string { - s := "" if r.Error == nil { - return s + return "" } - s = fmt.Sprintf("%s", r.Error.Error()) - return s + return r.Error.Error() } // fuzzCrashError is satisfied by a failing input detected while fuzzing. -- 2.50.0