use `machine` crate for state machines? about magic-wormhole.rs HOT 2 CLOSED

I also saw references to macro_machine and macro_machines.

machine itself gave me problems because I want to return a list of events from each transition (those events are then dispatched to other state machines, or they cause IO to happen), and machine didn't have a way to support this: the only return value was the new state instance.

I'm moderately happy with our current implementation: we use an Option<State> in each state machine, and remove the State object from the machine (with let oldstate = self.state.take().unwrap()) at the start of the process() method. Then a big match statement switches on the old state and returns the new state (along with pushing any actions/events that need to be returned to a nearby Vec). This way, we can consume the old State, along with anything it contained, and use those pieces to create the new State object. I'm still cloning things like Nameplates left and right, but maybe we can turn those into Rcs or something to reduce the cost a bit.

from magic-wormhole.rs.

I guess I'll close this for now. If anyone finds a cleaner way to implement our state machines, feel free to reopen it.

from magic-wormhole.rs.