[FFmpeg-devel] [PATCH] SubRip encoder

[Michael Niedermayer]

On Wed, Dec 22, 2010 at 12:56:48AM +0100, Aurelien Jacobs wrote:
> Hi,
> 
> Attached patch implements a SubRip subtitle encoder.
> The code used to parse ASS syntax and split it into basic elements
> is in a file of its own. It will be useful for any other text subtitle
> encoder.

It also will come in handy for a ass renderer :)


> 
> Aurel
>  Changelog              |    2 
>  MAINTAINERS            |    2 
>  libavcodec/Makefile    |    1 
>  libavcodec/allcodecs.c |    2 
>  libavcodec/ass_split.c |  468 +++++++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/ass_split.h |  172 ++++++++++++++++++
>  libavcodec/avcodec.h   |    2 
>  libavcodec/srtenc.c    |  301 +++++++++++++++++++++++++++++++
>  8 files changed, 946 insertions(+), 4 deletions(-)
> 66d4c3cf5b340d6581336e03af673ef767aab77f  srtenc.diff
> commit 4a4450b014df10ec6d17567a74b28d86cccd5433
> Author: Aurelien Jacobs <aurel at gnuage.org>
> Date:   Tue Sep 14 05:55:12 2010 +0200
> 
>     add SubRip encoder

lgtm

you could add some doxy for the internal functions though before commit

[....]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110107/b6232c2b/attachment.pgp>