Remove redundant tracing hooks from EVM.dryRun

onflow · Dec 31, 2024 · 99d8c10 · 99d8c10
1 parent 1326a7e
commit 99d8c10
Showing 1 changed file with 2 additions and 18 deletions.
diff --git a/fvm/evm/emulator/emulator.go b/fvm/evm/emulator/emulator.go
@@ -258,8 +258,6 @@ func (bl *BlockView) DryRunTransaction(
 	tx *gethTypes.Transaction,
 	from gethCommon.Address,
 ) (*types.Result, error) {
-	var txResult *types.Result
-
 	// create a new procedure
 	proc, err := bl.newProcedure()
 	if err != nil {
@@ -283,22 +281,8 @@ func (bl *BlockView) DryRunTransaction(
 	// we need to skip nonce check for dry run
 	msg.SkipAccountChecks = true
 
-	// call tracer
-	if proc.evm.Config.Tracer != nil && proc.evm.Config.Tracer.OnTxStart != nil {
-		proc.evm.Config.Tracer.OnTxStart(proc.evm.GetVMContext(), tx, msg.From)
-	}
-
-	// return without committing the state
-	txResult, err = proc.run(msg, tx.Hash(), tx.Type())
-
-	// call tracer on tx end
-	if proc.evm.Config.Tracer != nil &&
-		proc.evm.Config.Tracer.OnTxEnd != nil &&
-		txResult != nil {
-		proc.evm.Config.Tracer.OnTxEnd(txResult.Receipt(), txResult.ValidationError)
-	}
-
-	return txResult, err
+	// run and return without committing the state changes
+	return proc.run(msg, tx.Hash(), tx.Type())
 }
 
 func (bl *BlockView) newProcedure() (*procedure, error) {