Deduplicate state_provider_builder calls in payload validator

[shekhirin]

Summary

There are two implementations of state_provider_builder that could be deduplicated.

reth/crates/engine/tree/src/tree/mod.rs

Line 2483 in 4231f4b

match self.state_provider_builder(block_id.parent) {

reth/crates/engine/tree/src/tree/payload_validator.rs

Line 365 in 4231f4b

ensure_ok!(self.state_provider_builder(parent_hash, ctx.state()))

Solution

Instead of discarding the state_provider_builder here

reth/crates/engine/tree/src/tree/mod.rs

Line 2483 in 4231f4b

match self.state_provider_builder(block_id.parent) {

pass it as the argument to execute closure inside the ctx

reth/crates/engine/tree/src/tree/mod.rs

Line 2523 in 4231f4b

let executed = execute(&mut self.payload_validator, input, ctx)?;

to not create it again here

reth/crates/engine/tree/src/tree/payload_validator.rs

Lines 364 to 365 in 4231f4b

	let Some(provider_builder) =
	ensure_ok!(self.state_provider_builder(parent_hash, ctx.state()))

[Himess]

Can i take ?