BattleChain Confidence Pools

Summary

withdraw() calls _clampUserSums(msg.sender) but this call always returns immediately without doing anything. The withdraw gate already guarantees riskWindowStart == 0 before execution reaches that line, and _clampUserSums returns early when riskWindowStart == 0. The call is a dead code path that only wastes gas.

Root Cause

withdraw() gate ensures riskWindowStart == 0:

_clampUserSums immediately returns when riskWindowStart == 0:

Code snippet-
https://github.com/CodeHawks-Contests/2026-07-bc-confidence-pools/blob/58e8ba4ce3f3277866e4926f3140e597f9554a1e/src/ConfidencePool.sol#L305

Impact

Wastes gas on every withdrawal (extra SLOAD for riskWindowStart).

Fix

Simply remove the unnecessary call.