6975
89%

Ran 12 Oct 2026 12:06AM UTC

Jobs 1

Files 440

Run time 1min

Badge

Embed ▾

Committed 12 Oct 2026 12:02AM UTC coverage: 89.519%. Remained the same

Build # 6975

Build Type

push

github

Committed by

web-flow

Commit Message

refactor(string): remove unused UInt16 surrogate-pair helper (#4344)

`string/utils.mbt` carried a private copy of the surrogate-pair composition
that takes `UInt16` parameters. Nothing calls it: every real call site of
`code_point_of_surrogate_pair` lives in `builtin` (stringview, string_methods,
string, deprecated) and uses the `Int` version in `builtin/string.mbt`. The
only reference to this copy was its own test.

Besides being dead, the `UInt16` signature hides a difference. Its
`leading - 0xD800` is computed in `UInt16`, so it wraps modulo 0x10000 for
any `leading < 0xD800` instead of going negative the way the `Int` version
does. The two copies are bit-identical whenever `leading >= 0xD800`, which
is every input a guarded call site passes, so the divergence is invisible
today -- but it would change behaviour the moment the copies were unified,
or the `UInt16` one were reused by a caller holding raw code units (the
json lexer does exactly that).

Remove the function and its test. The remaining tests in the file cover the
`UInt16`/`Int` surrogate predicates and are unaffected.

Co-authored-by: SeekMoon <seekmoon@moonbitlang.com>

Coverage Stats

16740 of 18700 relevant lines covered (89.52%)

377688.57 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	6975.1	12 Oct 2026 12:07AM UTC	440	89.52	GitHub Action Run

moonbitlang / core / 6975
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6975

moonbitlang / core / 6975 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6975

moonbitlang / core / 6975
89%

README BADGES
x