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

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

2015-03-16 Thread James Almer
Signed-off-by: James Almer --- Better name (av_zero_high_bits?) and doxygen welcome. libavutil/common.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/libavutil/common.h b/libavutil/common.h index 852c1de..2ee5a98 100644 --- a/libavutil/common.h +++ b/libavutil/common.h @@ -