[FFmpeg-devel] [PATCH] frame: Simplify the video allocation

James Almer jamrial at gmail.com
Mon Sep 3 18:33:39 EEST 2018


On 9/3/2018 11:56 AM, James Darnley wrote:
> On 2018-09-03 15:29, James Almer wrote:
>> pass 32 - 1 to both av_image_fill_pointers() calls directly?
> 
> Please do not add a magic number where nobody will find it.  Use one of
> the 3 already existing methods for knowing the alignment necessary for
> assembly.

That magic number is STRIDE_ALIGN, and it's mentioned as such before and
after the patch. Except for the -1, which i don't know where it came
from. Michael (who i think added it) probably knows.

> 
> If this is unrelated, my apologies.
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 



More information about the ffmpeg-devel mailing list