From 544d36514a8aaaa7309cc398e3e23c1c8fc5e3c8 Mon Sep 17 00:00:00 2001 From: Andrew Bonventre Date: Sat, 27 Jan 2018 22:09:25 -0500 Subject: [PATCH] cmd/go/internal/test: remove redundant if statement Just return the result of the function call as they are both functionally equivalent. Change-Id: Ia7847c9b018475051bf6f7a7c532b515bd68c024 Reviewed-on: https://go-review.googlesource.com/90375 Run-TryBot: Andrew Bonventre TryBot-Result: Gobot Gobot Reviewed-by: Brad Fitzpatrick --- src/cmd/go/internal/test/test.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/cmd/go/internal/test/test.go b/src/cmd/go/internal/test/test.go index a99c6a5ec2..0a440585cb 100644 --- a/src/cmd/go/internal/test/test.go +++ b/src/cmd/go/internal/test/test.go @@ -1805,11 +1805,7 @@ func writeTestmain(out string, t *testFuncs) error { } defer f.Close() - if err := testmainTmpl.Execute(f, t); err != nil { - return err - } - - return nil + return testmainTmpl.Execute(f, t) } type testFuncs struct { -- 2.50.0