FAQ
Hi Jeff,

Thanks for this proposal. I don't think the performance gain here is
worth the extra code and extra bookkeeping overhead, so i'm -1 on this
proposal.

Cheers

Dave


https://codereview.appspot.com/6843056/diff/13001/src/pkg/net/http/request.go
File src/pkg/net/http/request.go (right):

https://codereview.appspot.com/6843056/diff/13001/src/pkg/net/http/request.go#newcode453
src/pkg/net/http/request.go:453: defer tpFree.put(tp)
The defer costs you an allocation so I don't think this is a net saving.

https://codereview.appspot.com/6843056/diff/13001/src/pkg/net/http/response.go
File src/pkg/net/http/response.go (right):

https://codereview.appspot.com/6843056/diff/13001/src/pkg/net/http/response.go#newcode107
src/pkg/net/http/response.go:107: defer tpFree.put(tp)
see previous

https://codereview.appspot.com/6843056/

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedNov 20, '12 at 9:18a
activeNov 20, '12 at 9:18a
posts1
users1
websitegolang.org

1 user in discussion

Dave: 1 post

People

Translate

site design / logo © 2021 Grokbase