[MPlayer-dev-eng] [PATCH] add support for subtitles thru lavf

[Diego Biurrun]

On Fri, Jul 13, 2007 at 02:29:45PM +0200, Aurelien Jacobs wrote:
> 
> The attached patch allows to display subtitles that are demuxed
> by lavf.
> Ok to apply ?

Didn't test the patch, but this is a very welcome feature!  One step
closer to replacing native demuxers...

> --- libmpdemux/demux_lavf.c	(r??vision 23766)
> +++ libmpdemux/demux_lavf.c	(copie de travail)
> @@ -432,6 +435,17 @@
> +        case CODEC_TYPE_SUBTITLE:{
> +            sh_sub_t* sh_sub;
> +            if(priv->sub_streams >= MAX_S_STREAMS)

nit: I prefer a space between if and the parentheses.

> @@ -612,6 +632,19 @@
>  
> +/** \brief Get the language code for a subtitle track.
> +
> +  Retrieves the language code for a subtitle track.
> +
> +  \param demuxer The demuxer to work on
> +  \param track_num The n'th subtitle track to get the language from

the subtitle track number to get the language from

Diego