Skip to content

fix: correct internal function reference in calc_excess_blob_gas comment#2866

Closed
Fibonacci747 wants to merge 1 commit intobluealloy:mainfrom
Fibonacci747:fix/blob-rs-comment-accuracy
Closed

fix: correct internal function reference in calc_excess_blob_gas comment#2866
Fibonacci747 wants to merge 1 commit intobluealloy:mainfrom
Fibonacci747:fix/blob-rs-comment-accuracy

Conversation

@Fibonacci747
Copy link
Copy Markdown
Contributor

Fix incorrect comment in calc_excess_blob_gas function

The comment incorrectly stated that the function uses calc_excess_blob_gas
internally, when it actually uses calc_excess_blob_gas_osaka.

rakita added a commit that referenced this pull request Aug 10, 2025
- PR #2866: Fix comment reference from calc_excess_blob_gas to calc_excess_blob_gas_osaka
- PR #2867: Remove debug macro from test code in inspector lib
- PR #2874: Replace len() == 0 with is_empty() for idiomatic Rust

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
rakita added a commit that referenced this pull request Aug 10, 2025
- PR #2866: Fix comment reference from calc_excess_blob_gas to calc_excess_blob_gas_osaka
- PR #2867: Remove debug macro from test code in inspector lib
- PR #2874: Replace len() == 0 with is_empty() for idiomatic Rust

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
@rakita rakita closed this Aug 10, 2025
@rakita
Copy link
Copy Markdown
Member

rakita commented Aug 10, 2025

Cleaned up here #2876

This was referenced Aug 8, 2025
lwedge99 pushed a commit to sentioxyz/revm that referenced this pull request Sep 16, 2025
…loy#2874 (bluealloy#2876)

- PR bluealloy#2866: Fix comment reference from calc_excess_blob_gas to calc_excess_blob_gas_osaka
- PR bluealloy#2867: Remove debug macro from test code in inspector lib
- PR bluealloy#2874: Replace len() == 0 with is_empty() for idiomatic Rust

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.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