Re: [FFmpeg-devel] [PATCH 1/2] libavutil: add av_zhb

2015-03-17 Thread James Almer
On 17/03/15 5:28 AM, Reimar Döffinger wrote: > On 17.03.2015, at 05:08, James Almer wrote: > >> Signed-off-by: James Almer >> --- >> Better name (av_zero_high_bits?) and doxygen welcome. > > Maybe av_wrap_intp2? (to align with clip function naming) > Or av_mod_p2? > Essentially it does a modulo

Re: [FFmpeg-devel] [PATCH 1/2] libavutil: add av_zhb

2015-03-17 Thread Reimar Döffinger
On 17.03.2015, at 05:08, James Almer wrote: > Signed-off-by: James Almer > --- > Better name (av_zero_high_bits?) and doxygen welcome. Maybe av_wrap_intp2? (to align with clip function naming) Or av_mod_p2? Essentially it does a modulo with a power of 2. Otherwise "clear" high bits seems a more