[FFmpeg-devel] [PATCH 4/8] Fix the size of workspace buffers in the motion pixels decoder.

Michael Niedermayer michaelni at gmx.at
Fri Sep 30 00:51:20 CEST 2011


On Fri, Sep 30, 2011 at 12:05:49AM +0200, fenrir at elivagar.org wrote:
> From: Laurent Aimar <fenrir at videolan.org>
> 
> Some buffers must be mod 4 in width and/or height.
[...]
>      MotionPixelsContext *mp = avctx->priv_data;
> +    int w4 = (avctx->width  + 3) & ~3;
> +    int h4 = (avctx->height + 3) & ~3;

could use FFALIGN(), not sure if it would be clearer or more confusing
to the average reader, thus patch applied,
thanks

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

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- 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/20110930/8da373c8/attachment.asc>


More information about the ffmpeg-devel mailing list