On Sun, Sep 21, 2014 at 08:38:58AM +0200, wm4 wrote: > On Sat, 20 Sep 2014 22:27:40 +0200 > Clément Bœsch <u...@pkh.me> wrote: > > > Just a patchset to ease the transition to a sane internal form for the > > decoded > > text subtitles. > > > > [PATCH 01/17] avcodec/jacosubdec: add some memory checks > > [PATCH 02/17] avcodec/microdvddec: add some memory checks > > [PATCH 03/17] avcodec/movtextdec: add some memory checks > > [PATCH 04/17] avcodec/mpl2dec: add some memory checks > > [PATCH 05/17] avcodec/samidec: add some memory checks > > [PATCH 06/17] avcodec/realtextdec: add some memory checks > > [PATCH 07/17] avcodec/srtdec: add some memory checks > > [PATCH 08/17] avcodec/subviewerdec: add some memory checks > > [PATCH 09/17] avcodec/textdec: add some memory checks > > [PATCH 10/17] avcodec/webvttdec: add some memory checks > > > > These are just a bunch of memory checks around ff_ass_add_rect() because I'm > > going to replace it by another function which will also require mem checks, > > so > > I'd better get it down now. > > Wouldn't it have been easier to add a version of ff_ass_add_rect() that > takes a bprint buffer? Although a function to conflates these different > tasks (checking the buffer and adding the subs) would have been > slightly ugly too. >
Right, Nicolas suggested something like that later in the thread, I will do that instead. -- Clément B.
pgpPjGpb2BwGj.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
