On Sun, Sep 21, 2014 at 11:20:26AM +0200, Nicolas George wrote: > Le jour de la Raison, an CCXXII, Clément Bœsch a écrit : > > --- > > libavcodec/webvttdec.c | 10 ++++++++-- > > 1 file changed, 8 insertions(+), 2 deletions(-) > > Patches 5-10 look right, except most of them could benefit from the > factorization. >
So I did that and pushed that part of the patchset, adjusted to use a bprint helper. Unfortunately, this raises a warning because av_bprint_is_complete() doesn't take a const parameter. I was too lazy to actually fix the API, but didn't want to cast to hide the warning. If you can send a patch to fix the bprint API that would be really appreciated. Thanks :) Thank you for your review, -- Clément B.
pgpWLUgDZYJzx.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
