[FFmpeg-devel] [PATCH 1/3] lavu/frame: add av_frame_check_align().

Nicolas George 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
built-in.

I suggest we get a third opinion on the matter.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170510/f47fd71f/attachment.sig>


More information about the ffmpeg-devel mailing list