clean up comments

Author: meyer9

crates/flashblocks/src/processor.rs

@@ -413,6 +413,10 @@ where
                     Some((receipt, state))
                 });
 
+                // If cached, we can fill out pending block data using previous execution results
+                // If not cached, we need to execute the transaction and build pending block data from scratch
+                // The `pending_blocks_builder.with*` calls should fill out the same data in both cases
+                // We also need to update the cumulative gas used and next log index in both cases
                 if let Some((receipt, state)) = cached_data {
                     let (deposit_receipt_version, deposit_nonce) = if transaction.is_deposit() {
                         let deposit_receipt = receipt
@@ -507,19 +511,13 @@ where
                             }) {
                                 Ok(receipt) => receipt,
                                 Err(ctx) => {
+                                    // This is a deposit transaction, so build the receipt from the context
                                     let receipt = alloy_consensus::Receipt {
-                                        // Success flag was added in `EIP-658: Embedding transaction status code
-                                        // in receipts`.
                                         status: Eip658Value::Eip658(ctx.result.is_success()),
                                         cumulative_gas_used: ctx.cumulative_gas_used,
                                         logs: ctx.result.into_logs(),
                                     };
 
-                                    // Cache the depositor account prior to the state transition for the deposit nonce.
-                                    //
-                                    // Note that this *only* needs to be done post-regolith hardfork, as deposit nonces
-                                    // were not introduced in Bedrock. In addition, regular transactions don't have deposit
-                                    // nonces, so we don't need to touch the DB for those.
                                     let deposit_nonce = (is_regolith_active
                                         && transaction.is_deposit())
                                     .then(|| {
@@ -535,11 +533,6 @@ where
                                     receipt_builder.build_deposit_receipt(OpDepositReceipt {
                                         inner: receipt,
                                         deposit_nonce,
-                                        // The deposit receipt version was introduced in Canyon to indicate an
-                                        // update to how receipt hashes should be computed
-                                        // when set. The state transition process ensures
-                                        // this is only set for post-Canyon deposit
-                                        // transactions.
                                         deposit_receipt_version: is_canyon_active.then_some(1),
                                     })
                                 }