[MPlayer-dev-eng] [PATCH] add optional device with dvd://
Mathieu SCHROETER
mathieu.schroeter at gamesover.ch
Tue Jul 8 14:39:15 CEST 2008
elupus a écrit :
> Mathieu SCHROETER <mathieu.schroeter <at> gamesover.ch> writes:
>
>> Hello,
>>
>> this patch will add an optional argument for stream_dvd to handle
>> a dvd_device with the MRL.. like cdda, vcd, etc, ...
>> Very useful when MPlayer is used with "-slave -idle" modes..
>>
>> dvd://[title|[start_title]-end_title][/device]
>>
>> An optional device can be already used with dvdnav://, but that is
>> not specified in the manpage. This patch will fix the manpage too..
>>
>> I'm not sure if that is the best way for a thing.. when dvd_close()
>> is called, there is no way in order to read the optional argument.
>> Then I've added a global variable with the current device used.
>> And if the argument is NULL, dvd_device is used as before..
>>
>> Regards,
>>
>> Attachment (stream_dvd-optional_device.patch): text/x-patch, 3697 bytes
>
> This may be abit late, but i find this counter intuative.
>
> Something like would have made more sence imho.
> dvd://[device/][title|[start_title]-end_title]
It needs a lot of changes.. and for consistency, changes
for cdd[ab], vcd too.
Look at the end of ./parser-mpcmd.c and I think you will
understand why (at least for dvd://)..
"/device" at the end is the simplest way IMO.
Regards,
Mathieu
More information about the MPlayer-dev-eng
mailing list