add TODO not to investigate weird behavior and unexpected test failure

This commit is contained in:
Jared Wasinger 2025-10-05 16:05:21 +08:00
parent 047b2569f6
commit 75bfcf27a9

View file

@ -2156,9 +2156,13 @@ func (bc *BlockChain) ProcessBlock(parentRoot common.Hash, block *types.Block, s
} }
ptime = time.Since(pstart) ptime = time.Since(pstart)
// TODO: remove. gross. shouldn't have to manually call Finalise // TODO: if I remove this check before executing balTracer.Finalise, the following test fails:
// to pull in post-tx-execution state changes to the BAL here. // ExecutionSpecBlocktests/shanghai/eip3855_push0/push0/push0_storage_overwrite.json
balTracer.Finalise() if constructBALForTesting {
// TODO: remove. gross. shouldn't have to manually call Finalise
// to pull in post-tx-execution state changes to the BAL here.
balTracer.Finalise()
}
// unset the BAL-creation tracer (dirty) // unset the BAL-creation tracer (dirty)
bc.cfg.VmConfig.Tracer = nil bc.cfg.VmConfig.Tracer = nil