moonbitlang / core / 4200

95%

push

github

refactor(builtin): use is_empty() for length() == 0 guards

8 mechanical sites in builtin/ (array.mbt, arrayview.mbt, fixedarray.mbt,
arraycore_js.mbt, bytes.mbt) replaced `self.length() == 0` (or
`separator.length() == 0`) with `.is_empty()`. The `is_empty()` method
body is itself `self.length() == 0`, so the compiled IR is identical
after inlining; this is purely a style consistency fix.

Same idiom recently merged for argparse (#3539). The wider codebase
(60+ uses) prefers `.is_empty()`.

Semantics-preserving: full `moon test` (6425/6425) passes and
`moon info` produces no `.mbti` change.

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

7 of 7 new or added lines in 4 files covered. (100.0%)

14835 of 15679 relevant lines covered (94.62%)

218861.96 hits per line