ruby-grape / grape / 26722632890

97%

master: 97%

Build Type

Pull #2753

github

Committed by ericproulx

Commit Message

Lazy-allocate ParamScopeTracker's identity hashes

ParamScopeTracker eagerly built two {}.compare_by_identity hashes in its
constructor, so every validating request allocated both even though many
validators (presence, coercion of flat params) never touch either tracker.

Replace the eager initialize with memoized private accessors that allocate
each hash on first use. Behaviour is unchanged: reads still return nil /
EMPTY_PARAMS when nothing was stored.

In a mixed-endpoint memory_profiler run this drops the two allocation sites
from ~1,900 to ~500 objects each (~0.72 MB / ~2,800 fewer objects overall).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #2753: Lazy-allocate ParamScopeTracker's identity hashes

Coverage Stats

1103 of 1193 branches covered (92.46%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

3531 of 3591 relevant lines covered (98.33%)

23292.46 hits per line