From 41d860cf0e7cb63003d55839b59949752b7fdecc Mon Sep 17 00:00:00 2001 From: mattyw Date: Thu, 28 Sep 2017 11:12:16 -0400 Subject: [PATCH] net/http: remove unused named return variables The existing implementation names a c net.Conn return which is never user. Leaving the returns unamed is marginally clearer. Change-Id: If9a411c9235b78c116a8ffb21fef71f7a4a4ce8f Reviewed-on: https://go-review.googlesource.com/66890 Reviewed-by: Brad Fitzpatrick --- src/net/http/server.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/net/http/server.go b/src/net/http/server.go index 4329e61e60..b59179fb3c 100644 --- a/src/net/http/server.go +++ b/src/net/http/server.go @@ -3161,10 +3161,10 @@ type tcpKeepAliveListener struct { *net.TCPListener } -func (ln tcpKeepAliveListener) Accept() (c net.Conn, err error) { +func (ln tcpKeepAliveListener) Accept() (net.Conn, error) { tc, err := ln.AcceptTCP() if err != nil { - return + return nil, err } tc.SetKeepAlive(true) tc.SetKeepAlivePeriod(3 * time.Minute) -- 2.48.1