[MPlayer-G2-dev] Re: module_list vs. gui
Alban Bedel
albeu at free.fr
Sat Aug 16 11:21:20 CEST 2003
Hi Arpi,
on Tue, 12 Aug 2003 20:27:47 +0200 you wrote:
> Hi,
>
> As I already found it, and tdoay Albeu asked too, there is a problem
> with the current config element type MODULE_LIST.
> The current code uses a function to find modules by name, but it's not
> useful for guis where you want to list the available modules.
>
> Let's see what are our options:
> - find_module_by_name() (current) - not good for gui (can't list
> modules)- pointer to module_t array - not good for code, there may be
> multiple
> lists, dinamically loadable modules (and as we discussed earlier, a
> CLI shouldnt load all those, only the explicitly named modules), and
> special hidden modules (for internal use, like null or vf_vo2/vf_vd
> wrappers)
> - enumeration func: a function which returns a single module at a call,
> and
> can be called until you get null, to get all available modules
> (you should pass the previous return value (or null at 1st call) as
> parameter, to make it thread safe / reentrant)
>
> I would go for a mix of the 1st and 3rd option, ie. an enumeration
> function which can also do searching. Something like the good old
> findfirst/findnext of DOS:
That's sound good to me.
> module_t* enum_modules(module_t* prev, char* mod_name, int type)
> prev: the result of previous call, NULL for 1st call
> mod_name: name of the module we're searching for (when explicitly
> named),
> or NULL (when we list all modules)
> type: some type value, like 0=static(builtin modules), 1=dinamic
> -1=internal
> (note: this field shouldn't be changed between the first and other
> calls, and implementations may read it only at 1st call)
> Mayeb it could be flags instead, ie 1, 2, 4, so they can be OR'ed
Flags would make much more sense imho.
Albeu
--
Everything is controlled by a small evil group
to which, unfortunately, no one we know belongs.
More information about the MPlayer-G2-dev
mailing list