[FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code
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
> instead of:
> Dialogue: Marked=0,0:02:42.42,0:02:44.15,Wolf
> (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
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?
More information about the ffmpeg-devel