#43 landed — the roster is a RECORD, instances at computed offsets #178
Loading…
Reference in a new issue
No description provided.
Delete branch "feat/178-43-landed-the-roster-is-a-record-instances-at"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
#43 landed — the roster is a RECORD, instances at computed offsets
Along the way:
Touches 34 files:
plan_sizes.md,src/rust/crates/ck_codegen/src/arg.rs,src/rust/crates/ck_driver/src/compile/fn_main.rs,src/rust/crates/ck_driver/src/compile/mod.rs,src/rust/crates/ck_driver/src/compile/placement.rs,src/rust/crates/ck_driver/src/compile/seat_resolve.rsand 28 more.Split. The decision in
each_placed_start_carries_its_roster_position_idis a pure function now, with its own tests insrc/rust/crates/ck_load/src/desugar/boot.rs.It runs per token. The buffer behind
plex_seatslives on the struct insrc/rust/crates/ck_load/src/loader.rsnow and is cleared rather than reallocated.Updated. The point about
src/rust/crates/ck_load/src/desugar/boot.rsturned out to be a real bug, so thank you.Two things inline. The one in
src/rust/crates/ck_load/src/desugar/boot.rsis the one I care about.@ -317,0 +425,4 @@/// binding uses, never a second one — and each range covers exactly the/// placeholder the driver settles.#[test]fn each_placed_start_carries_its_roster_position_id() {each_placed_start_carries_its_roster_position_idreads, decides and writes in one function. The deciding part is the piece I would want to test on its own. Worth splitting?@ -477,6 +492,7 @@ impl Ctx<'_> {actor_provides,receiver_binders,registered_actors,plex_seats: Vec::new(),plex_seatsallocates a fresh buffer per call. If this runs per frame or per token, can the buffer be reused?That is it for
src/rust/crates/ck_load/src/desugar/boot.rs. Thanks for the second pass.