ruby-grape / grape / 35524402103

99%

push

github

Stop allocating two match caches per route that Mustermann is never asked to use (#2973)

Mustermann::RegexpBased memoizes what it matched in two
ObjectSpace::WeakKeyMaps per pattern, one behind #match and one behind
#peek_match. Grape calls neither: the router matches through the
pattern's compiled #to_regexp, and #params -- the only method Grape
reaches Mustermann for on the request path, for a path it has to
percent-decode -- reads the captures straight off that regexp without
touching either map.

So every route carried two maps that were allocated and never read.
Passing cache: false leaves them out.

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

1464 of 1521 branches covered (96.25%)

Branch coverage included in aggregate %.

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

4325 of 4325 relevant lines covered (100.0%)

28772.42 hits per line