Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

encoding/binary: Add WriteVarint and WriteUvarint #12009

Closed
dsnet opened this issue Aug 4, 2015 · 2 comments
Closed

encoding/binary: Add WriteVarint and WriteUvarint #12009

dsnet opened this issue Aug 4, 2015 · 2 comments

Comments

@dsnet
Copy link
Member

dsnet commented Aug 4, 2015

Issue #2748 removed WriteVarint and WriteUvarint due to excessive allocations. With the introduction of io.ByteWriter, it is possible to implement these without the original performance penalty. It also makes the library more symmetrical.

@ianlancetaylor ianlancetaylor added this to the Unplanned milestone Aug 4, 2015
@gopherbot
Copy link

CL https://golang.org/cl/13094 mentions this issue.

@dsnet
Copy link
Member Author

dsnet commented Sep 29, 2015

Closing. See the discussion in CL/13094.

@dsnet dsnet closed this as completed Sep 29, 2015
@golang golang locked and limited conversation to collaborators Sep 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants