Skip to content

Comments

fix(handler): correct transaction ID decrement logic#2892

Merged
rakita merged 1 commit intobluealloy:mainfrom
jackwener:fix
Aug 17, 2025
Merged

fix(handler): correct transaction ID decrement logic#2892
rakita merged 1 commit intobluealloy:mainfrom
jackwener:fix

Conversation

@jackwener
Copy link
Contributor

No description provided.


// decrement transaction id as it was incremented when we discarded the tx.
acc.transaction_id -= acc.transaction_id;
acc.transaction_id -= 1;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

acc.transaction_id always is 0

@codspeed-hq
Copy link

codspeed-hq bot commented Aug 17, 2025

CodSpeed Performance Report

Merging #2892 will not alter performance

Comparing jackwener:fix (e6491d9) with main (6697f70)

Summary

✅ 173 untouched benchmarks

@rakita rakita merged commit 38c7e00 into bluealloy:main Aug 17, 2025
31 checks passed
@github-actions github-actions bot mentioned this pull request Aug 17, 2025
@jackwener jackwener deleted the fix branch August 18, 2025 02:14
@github-actions github-actions bot mentioned this pull request Aug 19, 2025
lwedge99 pushed a commit to sentioxyz/revm that referenced this pull request Sep 16, 2025
Co-authored-by: megakabi <jakevin@megatechnology.com>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 21, 2026
…2892)

Co-authored-by: megakabi <jakevin@megatechnology.com>
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