Re: [AD] datedit argument list

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


> Ok, I've put a lot of options to datedit_save_datafile in a struct for
> easier extension in the future. For now, it just contains all the fields
> as seperate integers (similar to the way they were passed to the file),

Ok, but you need to pass the structure by reference, not by value. Otherwise
the stack is loaded twice with the structure.

> but they could be combined in a single int as flags.

Not really, since three members are multiple-state variables.

> The patch updates both the grabber and the dat utility.

Commited with the above modification.

--
Eric Botcazou



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/