[MPlayer-cvslog] r21070 - trunk/m_config.c

Alban Bedel albeu at free.fr
Mon Nov 20 15:35:03 CET 2006


On Sun, 19 Nov 2006 18:41:31 +0100 (CET)
reimar <subversion at mplayerhq.hu> wrote:

> Fix obvious typo, although the exact effect is still unclear to me,
> see also bug #593.

> @@ -124,7 +124,7 @@
>        continue;
>      if(co->opt->flags & (M_OPT_GLOBAL|M_OPT_NOSAVE))
>        continue;
> -    if((co->opt->flags & M_OPT_OLD) && !(co->flags && M_CFG_OPT_SET))
> +    if((co->opt->flags & M_OPT_OLD) && !(co->flags & M_CFG_OPT_SET))
>        continue;
>      if(co->flags & M_CFG_OPT_ALIAS)
>        continue;

This is for emulating the behaviour of the old command line parser.
With the old parser options were reseted to their default value only if
they had been set on the command line. In other words if an option
wasn't set on the command line but the code changed it, it just stayed
as-is when going to the next file.
This was introduced because a few options relied on that behaviour back
then. Currently it seems no option use it, so that stuff should be just
be killed.

	Albeu



More information about the MPlayer-cvslog mailing list