Skip to content

Comments

fix: remove redundant U256::from on Host getters in instructions#3053

Merged
rakita merged 4 commits intobluealloy:mainfrom
Forostovec:fix/remove-redundant-u256-from-host-getters
Oct 6, 2025
Merged

fix: remove redundant U256::from on Host getters in instructions#3053
rakita merged 4 commits intobluealloy:mainfrom
Forostovec:fix/remove-redundant-u256-from-host-getters

Conversation

@Forostovec
Copy link
Contributor

  • Replace U256::from(context.host.block_number()) with context.host.block_number() in block_info.rs.
  • Replace U256::from(context.host.effective_gas_price()) with context.host.effective_gas_price() in tx_info.rs.
  • Aligns with Host methods already returning U256 and matches style of adjacent instructions.

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 6, 2025

CodSpeed Performance Report

Merging #3053 will not alter performance

Comparing Forostovec:fix/remove-redundant-u256-from-host-getters (1470e60) with main (07953cd)

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.

lgtm

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