[MPlayer-dev-eng] Fwd: Re: [PATCH] bug with title_start and title_end in the MRL dvd://

[Nico Sabbi]

Il giorno sab, 12/07/2008 alle 10.08 +0200, Nico Sabbi ha scritto:

> 
> ---------- Messaggio inoltrato ----------
> From: Mathieu SCHROETER <mathieu.schroeter at gamesover.ch>
> To: mplayer-dev-eng at mplayerhq.hu
> Date: Tue, 08 Jul 2008 14:58:22 +0200
> Subject: Re: [MPlayer-dev-eng] [PATCH] bug with title_start and
> title_end in the MRL dvd://
> Nico Sabbi a écrit :
>         On Tuesday 08 July 2008 13:56:48 Mathieu SCHROETER wrote:
>                 Mathieu SCHROETER a écrit :
>                         Hi,
>                         
>                         with dvd://,
>                         [[start_title]-end_title] are truncated to 2
>                         chars each one.
>                         
>                         for example:
>                         
>                          mplayer dvd://110-120
>                         
>                         plays title 11 and 12.. nevertheless,
>                         dvd_range() for a
>                         title is 1 to 255.
>                         
>                         
>                         This patch will fix the bug.
>                 better patch..
>                 
>                 entbuf is a table of 10 chars..
>                 
>                 
>                 Regards,
>                 Mathieu
>         
>         the truncation in the original code is wrong, but titles
>         range from 1 to 99, IIRC
> 
> I've not enough knowledges in this domain but with some researchs
> on the web.. you seem to be right..
> 
> In this case, this patch will fix at least the range when
> start_title and end_title. are used.
> 
> 
> *But* I don't know if the best is change this range or not because
> with dvd://200 (for example), 200 is not truncated (and not tested
> with this macro) and I've just a message like
> 
>  Invalid DVD title number: 200
> 
> from stream/stream_dvd.c.
> 
> 
> Regards,
> Mathieu
> 

fixed differently: now the hostname part accepts a range from 1 to 99