moonbitlang / core / 2432

96%

push

github

fix: Deque::drain should panic on invalid bounds

Change Deque::drain to use guard + abort for invalid inputs instead of
silently returning empty deque. This makes it consistent with Array::drain
behavior and helps catch programming errors early.

Panics if:
- start < 0
- start > self.length()
- len < 0 (when provided)
- start + len > self.length() (when len is provided)

The error messages include the actual values to aid debugging:
- "Deque::drain: start index out of bounds (start=X, deque.length()=Y)"
- "Deque::drain: len out of bounds (start=X, len=Y, deque.length()=Z)"

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

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

11111 of 11556 relevant lines covered (96.15%)

394074.5 hits per line