Skip to content

Commit f74ed44

Browse files
1911860538gopherbot
authored andcommitted
net/http/httputil: remove redundant pw.Close() call in DumpRequestOut
pw.Close() is already deferred earlier in DumpRequestOut. Change-Id: Icdc9366c816848ed0bb444022d2aa14e4ceaabac GitHub-Last-Rev: 7c38f9f GitHub-Pull-Request: golang#75029 Reviewed-on: https://go-review.googlesource.com/c/go/+/696415 Reviewed-by: Cherry Mui <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Damien Neil <[email protected]> Reviewed-by: Sean Liao <[email protected]> Auto-Submit: Sean Liao <[email protected]>
1 parent a9689d2 commit f74ed44

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

src/net/http/httputil/dump.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,6 @@ func DumpRequestOut(req *http.Request, body bool) ([]byte, error) {
147147

148148
req.Body = save
149149
if err != nil {
150-
pw.Close()
151150
dr.err = err
152151
close(quitReadCh)
153152
return nil, err

0 commit comments

Comments
 (0)