[MPlayer-dev-eng] [PATCH] First usable version of the new config system

Arpi arpi at thot.banki.hu
Wed Oct 30 20:58:28 CET 2002


Hi,

> Attached is a new version. The only 'big' remaining problem is with the
what about -dvd, -loop etc?

> x11 vo's
> (see my mail $SUBJ = "Potential bug in x11 based drivers ??"). I have 2 way to
> solve this : Add a flag to emulate the old config behaviour on some options
> (do not reverse them to default unless there were an user setting) or fix
i would prefer this

> x11 vo's
> to make sure that the needed vars are correctly set (wich would really
> solve the
> problem). I attached a patch that fix the fullscreen bug and -fixed-vo for xv.
> Is it ok this way ?
no!

the right fix would be the opt_ way, used at other places too.
ie, make opt_screenwidth, opt_screenheight, and set them from cfgparser.
then in vo layer init vo_screenwidth/vo_screenheight from that.

don't mess with vo_init()


A'rpi / Astral & ESP-team

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



More information about the MPlayer-dev-eng mailing list