[MPlayer-dev-eng] [PATCH] vo_macosx: fix device_id option after r28165

Guillaume POIRIER poirierg at gmail.com
Sun Jan 18 19:06:23 CET 2009


Hello,

On Fri, Jan 9, 2009 at 4:03 PM, Adrian Stutz <adrian at sttz.ch> wrote:
> I missed the screen_force BOOL in my patch to vo_macosx to replace the
> custom options parsing with subopt_parse().
>
> Because screen_force is never set to YES anymore, the device_id option has
> no effect on fullscreen.
>
> The attched patch fixes that and removes screen_force, since screen_id is
> only used to force fullscreen anyway. The patch instead allows screen_id to
> be negative for automatic fullscreen device selection and assumes a forced
> selection for any positive value.
>
> Patch also includes update of the manpage that includes the new negative
> value of device_id.

Tested and works, thus applied.

Guillaume

-- 
Only a very small fraction of our DNA does anything; the rest is all
comments and ifdefs.

Rita Rudner  - "When I eventually met Mr. Right I had no idea that his
first name was Always."



More information about the MPlayer-dev-eng mailing list