[MPlayer-G2-dev] Re: g2 config - restart...

[Arpi]

Hi,

> >The other struct is a very simple struct to keep all info we want to know
> >about a module/plugin/layer. It's inspired by Andriy's common generalized
> >plugin_info thingie. While his idea is basically good, he wanted to
> >generalize the whole instances types (like vf_instance_t), afair.
> 
> Do you mean all *_instance_t must be thing for application only, not
> unified?

yes
it's lib/layer dependent, and it's internal thing of layers.

> >so what about libdemux exporting plugin_info* demux_core_info and
> >plugin_info** demux_modules_info, and so on for every library.
> 
>     Fully agree with that. Only question - how application will import
> these infos? We will implement some registering function or we will see
> its just as arrays and let application integrate it all in own trees by
> itself?

yes.
they can call resgistering func for each libs/layers they use at startup or
just handle them so.
in a GUI, you don't have a single list of all plugins, but will have 
panels/windows, one for each layer (like: Video filter configuration panel)
so you need list of modules of one type. you don't even need to check for
types, you have one list for each type.

>     Other question is how we would implement adding of options from
> demux_core_info into common list of options? Why I asking that? If we
> have only main config_t structure then it's a static array so we cannot
> reallocate it. It's why I proposed to make a structure cfg_tree_t. Sure,
> it's a structure for layer 2 only and it will be used for parsing and
> searching, but it will be better than (re)allocate a memory for whole
> main config_t thing, won't it?

yes
it's the job of layer 3, what i'm working on (desigm. not code) right now
sure it will not be realloc, but some kind of tree.
i can imagien a single config_t which have links to other config_t structs
(even to the plugins) like subconfigs, with some new special types.


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu