Skip to content

chore: add debug assertions to set_action#2832

Merged
rakita merged 1 commit intobluealloy:mainfrom
DaniPopes:set_action-dbg
Aug 4, 2025
Merged

chore: add debug assertions to set_action#2832
rakita merged 1 commit intobluealloy:mainfrom
DaniPopes:set_action-dbg

Conversation

@DaniPopes
Copy link
Collaborator

@DaniPopes DaniPopes commented Aug 1, 2025

Calling set_action when an action is already set usually means that a previous halt or similar was not returned. Extracted from debugging #2778.

Calling `set_action` when an action is already set usually means
that a previous `halt` or similar was not returned.
@DaniPopes DaniPopes requested a review from rakita August 1, 2025 21:57
@codspeed-hq
Copy link

codspeed-hq bot commented Aug 1, 2025

CodSpeed Performance Report

Merging #2832 will not alter performance

Comparing DaniPopes:set_action-dbg (73d110a) with main (3ad3116)

Summary

✅ 171 untouched benchmarks

@rakita rakita merged commit 428ad97 into bluealloy:main Aug 4, 2025
30 checks passed
@DaniPopes DaniPopes deleted the set_action-dbg branch August 4, 2025 14:11
lwedge99 pushed a commit to sentioxyz/revm that referenced this pull request Sep 16, 2025
Calling `set_action` when an action is already set usually means
that a previous `halt` or similar was not returned.
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