[FFmpeg-devel] [PATCH 2/4] matroska: Mark S_TEXT/UTF-8 as Subrip encoded subtitles.

[Clément Bœsch]

On Sun, Aug 12, 2012 at 11:29:05PM +0200, Nicolas George wrote:
> Le sextidi 26 thermidor, an CCXX, Philip Langdale a écrit :
> > While not explicitly stated in the specs, the original author
> > has stated that S_TEXT/UTF-8 is expected to be text using Subrip
> > markup, but without Subrip in-band timing.
> > 
> > So, now that we have a decoder that conforms to this expectation,
> > let's use it.
> > 
> > Signed-off-by: Philip Langdale <philipl at overt.org>
> > ---
> >  libavformat/matroska.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavformat/matroska.c b/libavformat/matroska.c
> > index b386d74..5350af3 100644
> > --- a/libavformat/matroska.c
> > +++ b/libavformat/matroska.c
> > @@ -53,7 +53,7 @@ const CodecTags ff_mkv_codec_tags[]={
> >      {"A_VORBIS"         , AV_CODEC_ID_VORBIS},
> >      {"A_WAVPACK4"       , AV_CODEC_ID_WAVPACK},
> >  
> > -    {"S_TEXT/UTF8"      , AV_CODEC_ID_TEXT},
> > +    {"S_TEXT/UTF8"      , AV_CODEC_ID_SUBRIP},
> >      {"S_TEXT/UTF8"      , AV_CODEC_ID_SRT},
> >      {"S_TEXT/ASCII"     , AV_CODEC_ID_TEXT},
> >      {"S_TEXT/ASS"       , AV_CODEC_ID_SSA},
> 
> Now that you mention it (in another thread), we may want to be able to mux
> TEXT too, I think.
> 

This would first require an AV_CODEC_ID_TEXT encoder stripping all the
tags, right?

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120812/7b6ea94c/attachment.asc>