Kurosu a écrit :
> But let's concentrate for now on what this patch does. There are 3 steps:
> - recording actions, which requires some modification to the Action class
[...]
> - interfacing with it; basically, wherever network packets are sent, we
> store them to in the Replay buffer

After an IRC discussion with lodesi (Laurent DEFERT SIMONNEAU), we have
concluded that the Action should have a flag as to whom needs it. This
flag would indicate something like:
- replay needs it (=>Replay::StoreAction)
- clients need it (=>Network::SendAction)
- action handler (=>ActionHandler::NewAction)

This is would be passed manually to the constructor or by the way
actions are registered. This is unclear at the moment, as lodesi is
still working on the ActionHandler.

This code factorization will also hopefully reduce copy'n'paste bugs,
and address the repetitiveness mentioned by Romain BIGNON (aka progs on
IRC).

Kurosu

_______________________________________________
Wormux-dev mailing list
Wormux-dev@gna.org
https://mail.gna.org/listinfo/wormux-dev

Répondre à