[vlc-devel] Status of Win32 branch nightly

[Derk-Jan Hartman]

On 10 sep 2008, at 08:04, Marian Ďurkovič wrote:
> - we still have crash on AR change

Did we have a crashlog of this ?
We should really see if there is not a way to fix this.

> - after recent changes in ts demux, the telx module now always looks  
> for
> page 100. The broadcasters rarely declare ttxt subtitles in the TS,  
> so the ts
> demux only sees "Teletext initial page" in most cases. Thus I'd  
> suggest to
> autodetect ttxt subtitles by default from teletext, i.e. the  
> following patch:
>
> diff --git a/modules/codec/telx.c b/modules/codec/telx.c
> index 20b5a23..1b81d16 100644
> --- a/modules/codec/telx.c
> +++ b/modules/codec/telx.c
> @@ -77,7 +77,7 @@ vlc_module_begin();
>     set_subcategory( SUBCAT_INPUT_SCODEC );
>     set_callbacks( Open, Close );
>
> -    add_integer( "telx-override-page", -1, NULL,
> +    add_integer( "telx-override-page", 0, NULL,
>                  OVERRIDE_PAGE_TEXT, OVERRIDE_PAGE_LONGTEXT, true );
>     add_bool( "telx-ignore-subtitle-flag", 0, NULL,
>               IGNORE_SUB_FLAG_TEXT, IGNORE_SUB_FLAG_LONGTEXT, true );

Since telx only really can decode subs, I kinda like it. The problem  
is however that is breaks terribly for broadcasters with multiple  
(undeclared) languages and sometimes for pages with no background that  
are not subs (Now and later, newstickers etc) which are declared as  
subs, but are not.

DJ