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

implement DecodeWithMemTracking for frame pallets #7598

Merged
merged 5 commits into from
Feb 18, 2025

Conversation

serban300
Copy link
Contributor

Related to #7360

This PR implements DecodeWithMemTracking for the types in the frame pallets

The PR is verbose, but it's very simple. DecodeWithMemTracking is simply derived for most of the types. There are only 3 exceptions which are isolated into 2 separate commits.

@serban300 serban300 added R0-silent Changes should not be mentioned in any release notes T17-primitives Changes to primitives that are not covered by any other label. labels Feb 18, 2025
@serban300 serban300 self-assigned this Feb 18, 2025
@serban300 serban300 requested review from acatangiu, cheme and a team as code owners February 18, 2025 08:18
@serban300 serban300 force-pushed the decode-with-mem-tracking branch from 33a73e4 to 2bb0a0d Compare February 18, 2025 08:42
@bkchr bkchr enabled auto-merge February 18, 2025 12:37
@bkchr bkchr added this pull request to the merge queue Feb 18, 2025
Merged via the queue into paritytech:master with commit 67279ec Feb 18, 2025
233 of 239 checks passed
clangenb pushed a commit to clangenb/polkadot-sdk that referenced this pull request Feb 19, 2025
Related to paritytech#7360

This PR implements `DecodeWithMemTracking` for the types in the frame
pallets

The PR is verbose, but it's very simple. `DecodeWithMemTracking` is
simply derived for most of the types. There are only 3 exceptions which
are isolated into 2 separate commits.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T17-primitives Changes to primitives that are not covered by any other label.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants