[FFmpeg-devel] [PATCH 1/2] Kill timed SRT

Clément Bœsch u at pkh.me
Mon Sep 29 19:27:02 CEST 2014


On Mon, Sep 29, 2014 at 05:42:09PM +0200, Nicolas George wrote:
> L'octidi 28 fructidor, an CCXXII, Clément Bœsch a écrit :
> > ---
> >  libavcodec/srtdec.c       | 33 +++----------------------------
> >  libavcodec/srtenc.c       | 49 ++---------------------------------------------
> >  libavformat/matroska.c    |  1 -
> >  libavformat/matroskaenc.c | 45 +------------------------------------------
> >  libavformat/srtenc.c      | 11 ++++-------
> >  5 files changed, 10 insertions(+), 129 deletions(-)
> 
> I am in favour of doing this, and a quick glance at the code seems ok.
> 
> I have a small doubt about the handling of "move": it seems that with the
> timed srt case, \move(x,y) is translated into "X: Y:" in the timestamp line
> of the packet payload, while in the clean srt case it is dropped: should it
> not be translated into side data? That is not something changed by the
> patch, but the patch removes the "move" callback altogether, making the
> issue less obvious, if there is one.
> 

Ah you are right. But it's indeed not a regression, and fixing that would
mean to rework the subtitles encode API: it currently takes a buffer and
not an AVPacket, so the encoder is not able to set the side data...

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140929/863043c9/attachment.asc>


More information about the ffmpeg-devel mailing list