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.

Sign up for GitHub

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

Jump to bottom

[amino] Use pbbindings for MarshalAny. #200

Open
jaekwon opened this issue May 18, 2022 · 1 comment
Open

[amino] Use pbbindings for MarshalAny. #200

jaekwon opened this issue May 18, 2022 · 1 comment
Milestone
💡Someday/Maybe

Comments

@jaekwon
Copy link
Contributor

jaekwon commented May 18, 2022

MarshalAny() currently doesn't use pbbindings at all, so there are currently no proto savings for most values/nodes/types.
For reference, see the implementation of Marshal().
@moul moul added this to the 💡Someday/Maybe milestone Oct 20, 2022
@moul moul mentioned this issue Dec 2, 2022
Open
@moul
Copy link
Member

moul commented Dec 2, 2022

Copied here: tendermint/tendermint2#3

@moul moul moved this to 🔵 Not Needed for Launch in 🚀 The Launch [DEPRECATED] Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
🚀 The Launch [DEPRECATED]
Status: 🔵 Not Needed for Launch
Milestone
💡Someday/Maybe
Development

No branches or pull requests
2 participants
@jaekwon @moul