CyclopsMC / IntegratedTunnels / 33106868551

71%

master-1.21-lts: 70%

Pull #384

github

Avoid constructing block place contexts for unused replaceability checks

TunnelItemHelpers#placeItems and #pickUpItems, TunnelFluidHelpers#placeFluids
and ItemStorageBlockWrapper#getItemStacks eagerly called
BlockState#canBeReplaced with a freshly constructed BlockPlaceContext (plus a
BlockHitResult and a Vec3), even when the surrounding condition could never
use the result. That is the case on every tick of a world block or fluid
exporter or importer with the default ignoreReplacable = false.

These checks are now only evaluated when they can still influence the result.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SzSKkG1wgcUdBPY7ucN11A

611 of 1678 branches covered (36.41%)

Branch coverage included in aggregate %.

5791 of 7417 relevant lines covered (78.08%)

4.07 hits per line