Skip to content

Comments

feat: add generic state to ResultAndState#2897

Merged
rakita merged 1 commit intobluealloy:mainfrom
alessandromazza98:ale/add-state-generic-to-result-and-state
Aug 19, 2025
Merged

feat: add generic state to ResultAndState#2897
rakita merged 1 commit intobluealloy:mainfrom
alessandromazza98:ale/add-state-generic-to-result-and-state

Conversation

@alessandromazza98
Copy link
Contributor

@alessandromazza98 alessandromazza98 commented Aug 19, 2025

This small PR lets a third party inject a custom state to the ResultAndState type. The ExecResultAndState already supports for a generic state so it's really a small PR.

The alloy_evm::Evm trait relies on this ResultAndState type for the transact_raw method, so it's impossible to use a different state than the EvmState without this modification.

@rakita

@codspeed-hq
Copy link

codspeed-hq bot commented Aug 19, 2025

CodSpeed Performance Report

Merging #2897 will not alter performance

Comparing alessandromazza98:ale/add-state-generic-to-result-and-state (9339e8d) with main (6f9a57c)

Summary

✅ 173 untouched benchmarks

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

lgtm

@rakita rakita merged commit 8073236 into bluealloy:main Aug 19, 2025
31 checks passed
This was referenced Aug 19, 2025
@alessandromazza98 alessandromazza98 deleted the ale/add-state-generic-to-result-and-state branch August 19, 2025 15:48
lwedge99 pushed a commit to sentioxyz/revm that referenced this pull request Sep 16, 2025
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

Successfully merging this pull request may close these issues.

2 participants