On Wed, May 10, 2017 at 11:11:58PM +0200, Hendrik Leppkes wrote: > On Wed, May 10, 2017 at 11:10 PM, Nicolas George <geo...@nsup.org> wrote: > > 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. > > > > > > Perhaps, but its not used like that anywhere
it's used at least for the fft API afaik -- Clément B.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
