On Mon, Dec 31, 2018 at 02:50:26PM -0300, James Almer wrote: > On 12/31/2018 1:40 PM, Michael Niedermayer wrote: > > On Sun, Dec 23, 2018 at 07:24:49PM +0100, Michael Niedermayer wrote: > >> Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > >> --- > >> libavcodec/rangecoder.c | 19 +++++++++++++++++++ > >> libavcodec/rangecoder.h | 9 +++++++++ > >> libavcodec/tests/rangecoder.c | 7 +++++-- > >> 3 files changed, 33 insertions(+), 2 deletions(-) > > > > will apply > > Is this function called from a hot loop? If so, it may be worth making > it inline instead, seeing how the version argument in the only cases > it's used is a constant.
its only called per slice, so call overhead / an extra if() should not matter [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is what and why we do it that matters, not just one of them.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
