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

Ergzay ergzay at gmail.com
Thu Feb 26 21:08:33 CET 2009


On Jan 18, 2009, at 1:06 PM, Guillaume POIRIER wrote:

> 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.
>

I know I'm resurrecting an over a month old post but I thought I  
should say something.

While the current functionality is good with letting the user select  
the forced full screen, what should not be done is have the UI  
complain to the user that there is an error when there is not (if I am  
understanding this correctly). As currently with no device_id set,  
every time you run mplayer you get a "Get device error: Device ID -1  
do not exist, falling back to main device." This is not really an  
error, correct? A default setting shouldn't be an error setting.  
Shouldn't we instead remove said error (or change how it appears) or  
default the device ID to 0 (current display)?

Ergzay



More information about the MPlayer-dev-eng mailing list