Skip to content

Commit

Permalink
clarify log; rm todo
Browse files Browse the repository at this point in the history
  • Loading branch information
clabby committed Jan 17, 2025
1 parent cac8582 commit 757b174
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions bin/client/src/interop/transition.rs
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ where
H: HintWriterClient + Send + Sync + Debug + Clone,
{
// Check if we can short-circuit the transition, if we are within padding.
// TODO: Deduplicate claim check logic.
if let PreState::TransitionState(ref transition_state) = pre {
if transition_state.step >= transition_state.pre_state.output_roots.len() as u64 {
info!(
Expand Down Expand Up @@ -85,11 +84,11 @@ where
}

// In the case where the agreed upon L2 pre-state is the same as the claimed L2 post-state,
// trace extension is detected and we can skip the derivation and execution steps.
// the state transition is invalid as it does not extend the chain.
if boot.agreed_pre_state == boot.claimed_post_state {
info!(
target: "interop_client",
"Trace extension detected. State transition is already agreed upon.",
"No-op state transition is invalid; Pre == post.",
);
return Err(FaultProofProgramError::InvalidClaim(
boot.agreed_pre_state,
Expand Down

0 comments on commit 757b174

Please sign in to comment.