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

perf: internal objects optimization #281

Closed
wants to merge 1 commit into from
Closed

Conversation

iproudhon
Copy link
Contributor

perf: internal objects optimization

  1. replaced cdc.(Un)MarshalInterface() by protobuf (Un)Marshal for internal objects: BaseAccount, Balance & Supply
  2. PubKey in BaseAccount is secp256k1.PubKey, not interface, to avoid 'Any' type handling
  3. address fields in the above objects and MsgSend (hence tx as well) are no longer strings, but sdk.AccAddress

…sgSEnd)

1. replaced cdc.(Un)MarshalInterface() by protobuf (Un)Marshal for internal objects: BaseAccount, Balance & Supply
2. PubKey in BaseAccount is secp256k1.PubKey, not interface, to avoid 'Any' type handling
3. address fields in the above objects and MsgSend (hence tx as well) are no longer strings, but sdk.AccAddress
@CLAassistant
Copy link

CLAassistant commented Jul 13, 2021

CLA assistant check
All committers have signed the CLA.

@iproudhon iproudhon assigned iproudhon and unassigned egonspace Jul 13, 2021
@iproudhon
Copy link
Contributor Author

Some test files were not updated.

@iproudhon iproudhon closed this Jul 13, 2021
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.

3 participants