Change Details

and remove an unnecessary g_avalanche check in `Chainstate::FindMostWorkChain`: the callsites already ignore `blocksToReconcile` if the processor is not initialized (which only happens in unit tests), so there is no significant change in behavior. This concludes the removal of g_avalanche access in Chainstate functions. Depends on D16313

Pass a reference rather than a pointer, so that this function only ever gets called if there is an avalanche processor initialized. This concludes the removal of g_avalanche access in Chainstate functions. Depends on D16313

and remove an unnecessary g_avalanche check in `Chainstate::FindMostWorkChain`: the callsites already ignore `blocksToReconcile` if the processor is not initialized (which only happens in unit tests)Pass a reference rather than a pointer, so there is no significant change in behaviorat this function only ever gets called if there is an avalanche processor initialized. This concludes the removal of g_avalanche access in Chainstate functions. Depends on D16313