[FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

Rainer Hochecker fernetmenta at online.de
Mon Sep 22 20:55:54 CEST 2014


Clément Bœsch <u <at> pkh.me> writes:

> 
> On Mon, Sep 22, 2014 at 03:12:02PM +0000, Rainer Hochecker wrote:
> [...]
> > Hi,
> > 
> > Thank you very much for this explanation and sorry for not having read
> > release notes properly. subs are not my territory but the maintainer is busy
> > with real life so I will try to adapt to those changes.
> > 
> 
> Feel free to ask more direct questions.
> 
> Basically, the idea is that the matroska demuxer will now output
> AVPacket.data that looks like this:
> 
>     1,,Wolf main,Cher,0000,0000,0000,,Et les enregistrements de ses ondes
delta ?
> 
> instead of:
> 
>     Dialogue: Marked=0,0:02:42.42,0:02:44.15,Wolf
main,autre,0000,0000,0000,,Toujours rien.
> 
> (to take the matroska specs example)
> 
> The later had several problems, like including timestamp in a string which
> couldn't be altered without insane hacks at format level. It also dropped
> the ReadOrder fields which is needed to get proper rendering.
> 
> [...]
> 
> Note that we still use the "Dialogue: ..." form for internal decoded form
> of the subtitles (what you get into AVSubtitle when calling the decode
> subtitles function on the AVPacket), but I'm working on fixing this. I'll
> try to document that change even more, and compatibility will last for a
> long time again.
> 

I only needed to add AV_CODEC_ID_ASS to our handler which already took
AV_CODEC_ID_SSA.
According to the discussion here:
http://lists.ffmpeg.org/pipermail/ffmpeg-devel/2014-May/157170.html, is it
correct to drop AV_CODEC_ID_SSA?

Regards,
Rainer





More information about the ffmpeg-devel mailing list