[MPlayer-dev-eng] [PATCH] -noconfig option
Ivan Kalvachev
ikalvachev at gmail.com
Sun Apr 13 13:28:04 CEST 2008
On Sat, Apr 12, 2008 at 7:17 PM, Andrew Savchenko <Bircoph at list.ru> wrote:
> Hello,
> and thank you for reply.
>
>
> On Saturday 12 April 2008 12:57, Alban Bedel wrote:
> > > int m_config_parse_config_file(m_config_t* config, char
> > > *conffile) {
> > > + if (noconfig)
> > > + return 1;
> >
> > I don't like that very much. Having a global simply disable a
> > function is ugly. Better check before looking for config files.
> > It's a bit more work but you could also allow finer disabling
> > (to only use the user's config file for ex.).
>
> Done. Some code lookup shows that there is 5 different types of
> config files:
> 1) system-wide;
> 2) user-specific;
> 3) include;
> 4) user-filedir;
> 5) gui.
>
> So I introduced 4 new options:
> -noconfig, -disable-system-conf, -disable-user-conf, and
> -disable-gui-conf (gui only). Option -noconfig is equivalent to
> all -disable-*-conf specified simaltaneously.
How about having only one option -noconfig that takes combination of
parameters "system","user","include","filedir","gui", "all" ?
More information about the MPlayer-dev-eng
mailing list