[MPlayer-dev-eng] [PATCH] libass support in demux_lavf

Evgeniy Stepanov eugeni.stepanov at gmail.com
Sat Dec 29 23:21:34 CET 2007


On Sunday 30 December 2007 00:38:26 Diego Biurrun wrote:
> On Sun, Dec 30, 2007 at 12:15:26AM +0300, Evgeniy Stepanov wrote:
> > this patch adds libass support for SSA/ASS tracks in libavformat demuxer.
> > Depends on ffmpeg r11352. Embedded fonts are not supported yet.
>
> Please do not use tabs in a file that contains only spaces for
> indentation.

Which does not apply to demux_lavf.c :)

> > --- libmpdemux/demux_lavf.c	(revision 25540)
> > +++ libmpdemux/demux_lavf.c	(working copy)
> > @@ -396,13 +401,21 @@
> >              if(priv->sub_streams >= MAX_S_STREAMS)
> >                  break;
> >              /* only support text subtitles for now */
> > -            if(codec->codec_id != CODEC_ID_TEXT)
> > +            if(codec->codec_id != CODEC_ID_TEXT &&
> > +	       codec->codec_id != CODEC_ID_SSA)
>
> This could be nicely aligned.

Anyway, here is untabified and reindented patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: demux_lavf.patch
Type: text/x-diff
Size: 2975 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20071230/521376fd/attachment.patch>


More information about the MPlayer-dev-eng mailing list