Skip to content

Comments

refactor(inspector): remove unnecessary mutable borrow in GasInspector::step_end#3035

Merged
rakita merged 2 commits intobluealloy:mainfrom
Bilogweb3:refactor/inspector-step_end-immutable-gas
Oct 1, 2025
Merged

refactor(inspector): remove unnecessary mutable borrow in GasInspector::step_end#3035
rakita merged 2 commits intobluealloy:mainfrom
Bilogweb3:refactor/inspector-step_end-immutable-gas

Conversation

@Bilogweb3
Copy link
Contributor

I updated GasInspector::step_end to take &Gas (not &mut Gas) because it only reads, reducing borrow friction and fixing Clippy warnings with no behavior change

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 1, 2025

CodSpeed Performance Report

Merging #3035 will not alter performance

Comparing Bilogweb3:refactor/inspector-step_end-immutable-gas (91af78f) with main (6e29b0c)

Summary

✅ 173 untouched

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.

makes sense

@rakita rakita merged commit c56206a into bluealloy:main Oct 1, 2025
31 checks passed
@github-actions github-actions bot mentioned this pull request Oct 5, 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