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

explore removing total length from PackedCounter #1420

Open
holic opened this issue Sep 8, 2023 · 0 comments
Open

explore removing total length from PackedCounter #1420

holic opened this issue Sep 8, 2023 · 0 comments

Comments

@holic
Copy link
Member

holic commented Sep 8, 2023

if we remove total length, we could get at least one more dynamic field, and simplifies encoding/decoding

I think we currently use it to preallocate memory for loading data from dynamic fields, so we'd need to replace it with a thing that sums up each length? we should see if it's worth the gas trade-off for the extra field

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

No branches or pull requests

1 participant