Skip to content

Conversation

@paulhauner
Copy link
Member

Issue Addressed

NA

Proposed Changes

Avoids lots of grow allocations when decoding a VariableList of fixed-length items. This is the function used for decoding the state.validators list.

Additional Info

NA

@paulhauner paulhauner added the ready-for-review The code is ready for review label Nov 17, 2020
Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tasty 👌

@paulhauner
Copy link
Member Author

Thanks!

bors r+

@paulhauner paulhauner added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Nov 17, 2020
bors bot pushed a commit that referenced this pull request Nov 17, 2020
## Issue Addressed

NA

## Proposed Changes

Avoids lots of grow allocations when decoding a `VariableList` of fixed-length items. This is the function used for decoding the `state.validators` list.

## Additional Info

NA
@bors
Copy link

bors bot commented Nov 17, 2020

@bors bors bot changed the title Avoid allocations on VariableList [Merged by Bors] - Avoid allocations on VariableList Nov 17, 2020
@bors bors bot closed this Nov 17, 2020
@paulhauner paulhauner deleted the alloc-reduce branch March 17, 2021 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants