- Modules: `use`/`pub`, directory-as-module, per-module symbol resolution (a
flat first-wins merge silently ran the wrong `pub fn` body), six reserved
stdlib namespaces typed UNKNOWN-BUT-RESERVED.
- Surface: `and`/`or` (own precedence tier, short-circuit, Bool-only), `${}`
interpolation desugared at parse time, `const`, break/continue with
drop-correct exits, do-while, inline-fn rejection.
- switch expr/stmt: required `default` over scalars/Text, arm unification,
EQ/EQS+JZ lowering, per-arm drop scopes with N-way JOIN-DROP; `default`
sorted last by a shared lowering order (textual order made arms dead).
- typedef records: structural, same shape = one class entry; `?name: T`
nullable-by-shape; emit_ctor fills omitted defaults; `type` as field name.
- Enum variants: all-bare unions = int ordinals; any-payload = one class
entry per variant, tag IS the header class_id (no header field, no format
bump); exhaustive switch without `default`; arity checked both directions.
- Payload escape modeled as move-out (pointer-kind fields only — a scalar
escape is a copy); caller reaps owned heap temps passed by borrow: two
unbounded LSan-blind leaks, 10.5 MB -> 1.5 MB flat over 300k iterations.
- Fixed en route, each with a RED repro: dead E209 builtin-arg check and
`int_to_text` missing from both types.ml builtin tables (both segfaulted
wovm), multi-file phantom double-report, emit_ctor's field temp clobbering
dst in tail position (pre-existing), warnings swallowed without an error.
- Two fenced VM builtins: `int_to_text` (13), `variant_tag` (14).
- 14+565 unit (was 14+401), corpus 71 (was 32) plain and under wovm_asan,
wovm-test + cli_smoke green. Log-watcher 307 -> 93 diagnostics (85 E101 /
4 E207 / 1 E208 / 3 W202); the 5 non-E101 residuals await Task 7 grammar.
27 lines
1 KiB
Text
27 lines
1 KiB
Text
-- haxe-parity Task 3, review fix (Critical 1): `default` textually
|
|
-- BEFORE some `case` arms used to silently kill every arm after it --
|
|
-- `default` has no comparison of its own, so lowering the arms in raw
|
|
-- source order meant nothing ever jumped into a `case` written after
|
|
-- it, and `default`'s own body jumped straight past it to the
|
|
-- switch's exit. Reviewer-reproduced repro shape, verbatim: with the
|
|
-- bug, `classify(2)` returned "many" (default's own value) instead of
|
|
-- "two". Fixed by lowering `default` last regardless of source
|
|
-- position (ast.ml's `switch_lowering_order`); this fixture pins the
|
|
-- corrected *runtime* behavior (a `--dump-owner`/typecheck assertion
|
|
-- can't see this — only actually running the compare-and-jump chain
|
|
-- proves the arm was reachable again).
|
|
fn classify(n: Int) -> Text {
|
|
let v = switch n {
|
|
default: "many";
|
|
case 1: "one";
|
|
case 2: "two";
|
|
};
|
|
return v;
|
|
}
|
|
|
|
fn main() -> Int {
|
|
print(classify(1));
|
|
print(classify(2));
|
|
print(classify(3));
|
|
return 0;
|
|
}
|