moonbitlang / core / 4761

91%

push

github

refactor(json): use match in the hot number-scan and whitespace loops

Convert the two remaining char `if`/`else if` cascades in the lexer's
hot loops to `match`, finishing the pattern started in the merged
char-range refactors:

- scan_json_number: `if c is ('0'..='9') … else if c == '.' … else if
  c == 'e' || c == 'E'` becomes a `match` with `'0'..='9' as c`, `'.'`
  and `'e' | 'E'` arms (digit case still first).
- lex_skip_whitespace: `c == ' ' || c == '\t' || c == '\r' || c == '\n'`
  becomes a `' ' | '\t' | '\r' | '\n'` match arm.

Both sit on the JSON parse hot path, so this was benchmark-checked
(`moon bench -p json --target native --release`, n=1000 mixed
object/array). Interleaved A/B over the working tree shows the match
form is neutral-to-slightly-faster, never slower (e.g. 1.00 ms → ~0.95
ms in one block, parity in another). No behavior change; 175/175 tests
pass.

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

20 of 20 new or added lines in 2 files covered. (100.0%)

15151 of 16656 relevant lines covered (90.96%)

197437.6 hits per line