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 jamr...@gmail.com wrote: Signed-off-by: James Almer jamr...@gmail.com --- 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.

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 jamr...@gmail.com wrote: Signed-off-by: James Almer jamr...@gmail.com --- Better name (av_zero_high_bits?) and doxygen welcome. Maybe av_wrap_intp2? (to align with clip function naming) Or av_mod_p2?

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

2015-03-16 Thread James Almer
Signed-off-by: James Almer jamr...@gmail.com --- 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 +++