search

osmosis-labs / cosmos-sdk

:chains: A Framework for Building High Value Public Blockchains :sparkles:
https://cosmos.network/
Apache License 2.0
14 stars 34 forks source link

Quick speedup for coins.String() #394

Closed ValarDragon closed 1 year ago

ValarDragon commented 1 year ago

What is the purpose of the change

Minor speedup for coins.String(), to reduce allocations for this. This did appear in benchmarks for how long our tests take. (Its 25% of the SendCoins time, which itself can be a lot depending on the test)

github-actions[bot] commented 1 year ago

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.