-
Couldn't load subscription status.
- Fork 374
call process_ethereum_block from monad #1824
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: alee/runloop
Are you sure you want to change the base?
Conversation
5331f28 to
4834461
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR refactors block execution logic by extracting revert transaction generation into a separate module. The primary goal is to improve code modularity by separating Ethereum-specific block processing logic from Monad-specific consensus handling.
Key changes:
- Introduces a new
revert_transaction_generatormodule that encapsulates the logic for creating revert transaction functions based on block cache and chain context - Refactors
process_ethereum_blockto accept a revert transaction generator as a parameter, making it more generic and reusable - Moves
BlockCacheEntryandBlockCachetype definitions fromrunloop_monad.cppto the new module
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| cmd/monad/revert_transaction_generator.hpp | Defines new types and function signature for revert transaction generation |
| cmd/monad/revert_transaction_generator.cpp | Implements the revert transaction generator with block cache management and chain context setup |
| cmd/monad/runloop_ethereum.hpp | Adds forward declarations and function signature for process_ethereum_block with revert transaction generator parameter |
| cmd/monad/runloop_ethereum.cpp | Refactors process_ethereum_block to accept and use revert transaction generator; moves database finalization to caller |
| cmd/monad/runloop_monad.cpp | Simplifies propose_block by delegating execution to process_ethereum_block with a generated revert transaction function |
| cmd/CMakeLists.txt | Adds new source and header files to the build configuration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| i, | ||
| state, | ||
| chain_context); | ||
| return false; |
Copilot
AI
Oct 23, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unreachable code detected. The function returns on line 95-103, making line 104's return false; statement unreachable and dead code.
| return false; |
4834461 to
e83b231
Compare
69de864 to
da8c92e
Compare
5a82b0d to
c1269fc
Compare
c1269fc to
c42d926
Compare
No description provided.