[FFmpeg-devel] in preparation of changing the internal form of decoded text subtitles

[Clément Bœsch]

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 at 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.
-------------- 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/20140921/2c1f258a/attachment.asc>