Martin Quinson authored
- Use std::vector instead of C arrays (old API remains but is not documented -- it should be properly deprecated) - The flop_amounts and comm_amounts arrays are not automatically freed by the internal functions, and should be properly cleaned by their creator. - EXCEPTION: in ptask model, sequential exec and regular comms don't have a real caller, so the internal function still has to free these arrays... Sick Sad World. - The proper solution would be to have the only copy of these arrays in the Action instead of having it in s4u. - But for now, Actions start as soon as created. So if you want to init them without starting, you have to have the data in s4u and only create the implementation side when you start the stuff. - That should obviously be fixed :) First step in that direction would be to have the constructor of each action NOT register the action in the LMM, but have an Action::start() in charge of this. For each subclass of Action.