kyren / gc-arena

Incremental garbage collection from safe Rust
Creative Commons Zero v1.0 Universal
438 stars 36 forks source link

Remove Phase::Wake #20

Closed Aaron1011 closed 1 year ago

Aaron1011 commented 1 year ago

We now transition directly from Phase::Sleep to Phase::Propagate, pushing the root pointer to the gray queue. Propagation will treat the root as an ordinary GcBox allocation.

kyren commented 1 year ago

This looks good to me, and makes perfect sense. The 'Wake' phase has always been very awkward. Sorry for taking a while to review this!