[FFmpeg-devel] [PATCH 1/3] lavu/frame: add av_frame_check_align().
george at nsup.org
Thu May 11 00:10:48 EEST 2017
Le primidi 21 floréal, an CCXXV, Hendrik Leppkes a écrit :
> Everywhere I found where the align value is used, its used as (1 <<
> alignment). In that case, I would prefer to pass the actual alignment
> here (ie. 32 instead of 5), which is an easier value to understand and
> matches the various alignment constants/values we already had before.
I can live with that, but here is my rationale for doing it that way:
log2(align_mult) is a non-trivial function while 1<<align_log is a CPU
I suggest we get a third opinion on the matter.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 833 bytes
Desc: Digital signature
More information about the ffmpeg-devel