moonbitlang / core / 4094

94%

push

github

fix(array): abort on negative repeat count and guard against overflow

`Array::repeat(-1)` previously raised `RuntimeError: requested new array
is too large`, which is opaque and points the user at allocation rather
than at the bad argument. Negative `times` is almost always a caller
bug (sign error / off-by-one), so abort with `"negative repeat count"`
instead. A `times` of `0` still returns an empty array.

Also mirror the multiplicative-overflow guard already present in
`Bytes::repeat` so a wrap-around in `len * times` aborts cleanly with
`"repeat result too large"` instead of silently producing a wrong-sized
capacity hint.

Refs #3498. Split out from #3505 for easier review.

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

5 of 6 new or added lines in 1 file covered. (83.33%)

14833 of 15678 relevant lines covered (94.61%)

218871.76 hits per line