]> Cypherpunks repositories - gostls13.git/commitdiff
net/http: remove unused named return variables
authormattyw <gh@mattyw.net>
Thu, 28 Sep 2017 15:12:16 +0000 (11:12 -0400)
committerBrad Fitzpatrick <bradfitz@golang.org>
Thu, 2 Nov 2017 21:43:30 +0000 (21:43 +0000)
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 <bradfitz@golang.org>
src/net/http/server.go

index 4329e61e609515d6490d38bd5ea177f1b81dd291..b59179fb3c506f318ed29b2b072168f4d9cdd04f 100644 (file)
@@ -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)