moonbitlang / core / 4259

94%

push

github

refactor: build to_array result without mutation

Two more sites collapse to a comprehension, mirroring the to_json sweep
in #3547:

- hashset/hashset.mbt — `HashSet::to_array`:
  collapse `let arr = Array::new(...); for entry in self.entries { if ... { arr.push(...) } }`
  to `[for entry in self.entries if entry is Some({key, ..}) => key]`.
- sorted_map/map.mbt — `SortedMap::to_array`:
  collapse `let arr = Array::new(...); self.each((k, v) => arr.push((k, v)))`
  to `[for k, v in self => (k, v)]`.

Both `to_array` methods are conversion paths, not on the matching/insertion
hot path of the underlying data structure.

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>

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

14877 of 15720 relevant lines covered (94.64%)

218296.93 hits per line