[MPlayer-dev-eng] [PATCH] Change lots of defines to one big enum.

Ivan Kalvachev ikalvachev at gmail.com
Tue Jun 17 22:59:23 CEST 2008


On 6/17/08, Benjamin Zores <ben at geexbox.org> wrote:
> Hi,
>
> Looking at this input/input.h file, I've found it pretty ugly to use
> dozens of defines to declare MPlayer type of commands.
> This simple patch makes them a big enum which:
>   - ease a lot application of external patches (no command number conflict)
>   - allows sorting commands per type/group (let's call this the Diego's
> work)
>   - order command alphabetically per group (Diego, if you read this and
> care ... :-)
>
> Anyone objects to this or may I apply ?

I like the idea.

I just wonder why are you making it into typedef
(are you planning or replacing current int with that type?)
and won't the mp_command_t be confused with mp_cmd_t that
already exists.

Best Regards



More information about the MPlayer-dev-eng mailing list