[FFmpeg-devel] [PATCH 3/7] utvideoenc: use ff_generate_len()

Michael Niedermayer michaelni at gmx.at
Wed Aug 22 17:25:58 CEST 2012


On Wed, Aug 22, 2012 at 10:52:00AM -0400, Derek Buitenhuis wrote:
> On 22/08/2012 10:26 AM, Michael Niedermayer wrote:
> > 19% faster
> > smaller files
> > this may also fix possible integer overflows due to previous 32bit useage
> > 
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> 
> LGTM, as long as someone verifies that these decode correctly with utvideodec.c,
> aswell as the upstream decoder.

tested, applied

thanks

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

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120822/e720f484/attachment.asc>


More information about the ffmpeg-devel mailing list