Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Expose storage_prefix logic, and remove duplicate code #9621

Merged
6 commits merged into from
Aug 25, 2021

Conversation

shawntabrizi
Copy link
Member

This exposes the logic behind generating a storage_prefix, and also removes lots of duplicated code where this function can be used instead.

@shawntabrizi shawntabrizi added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Aug 24, 2021
Co-authored-by: Guillaume Thiolliere <[email protected]>
@shawntabrizi
Copy link
Member Author

bot merge

@ghost
Copy link

ghost commented Aug 25, 2021

Waiting for commit status.

@ghost ghost merged commit 17ce41a into master Aug 25, 2021
@ghost ghost deleted the shawntabrizi-storage-prefix branch August 25, 2021 19:16
Wizdave97 pushed a commit to Wizdave97/substrate that referenced this pull request Aug 25, 2021
…9621)

* expose storage prefix generation, remove duplicate code

* remove more duplicate code

* clean up import

* fix io test

* remove slicing

* Update frame/support/src/storage/mod.rs

Co-authored-by: Guillaume Thiolliere <[email protected]>

Co-authored-by: Guillaume Thiolliere <[email protected]>

fix: revert unneccessary changes
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants