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

Enhance Encoder #14

Closed
wants to merge 2 commits into from
Closed

Enhance Encoder #14

wants to merge 2 commits into from

Conversation

yznima
Copy link
Contributor

@yznima yznima commented Dec 2, 2018

  • Write vector type and attr for all lists in a common codepath
  • Common path for all the -KB, -UU, -KG, -KH, -KI, -KJ, -KE, -KF, -KC, -KM, -KZ, -KN, -KU, -KV, KB, UU, KG, KH, KI, KJ, KE, KF, KM, KZ, KN, KU, KV types
  • Avoid extra allocation by casting strings to byte. Use WriteStrings instead

Needs to go after #10

@yznima yznima changed the title [WIP] Enhance Encoder Enhance Encoder Feb 11, 2019
@yznima
Copy link
Contributor Author

yznima commented Feb 11, 2019

@sv Could you review?

@yznima yznima closed this Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant