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

Add StorageIndex newtype #2332

Open
sistemd opened this issue Oct 23, 2024 · 0 comments
Open

Add StorageIndex newtype #2332

sistemd opened this issue Oct 23, 2024 · 0 comments
Labels
good first issue Good for newcomers

Comments

@sistemd
Copy link
Contributor

sistemd commented Oct 23, 2024

In our database, every trie node has a unique storage index. We simply represent this as a u64 in the code. The code would be a bit clearer and a bit less error prone if we had a simple newtype for it.

@sistemd sistemd self-assigned this Oct 23, 2024
@sistemd sistemd assigned sistemd and unassigned sistemd Oct 31, 2024
@sistemd sistemd added the good first issue Good for newcomers label Nov 12, 2024
@sistemd sistemd removed their assignment Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant