[FFmpeg-devel] [PATCH] intmath: remove av_ctz.

James Almer jamrial at gmail.com
Sun Oct 11 21:19:01 CEST 2015


On 10/11/2015 4:16 PM, Timothy Gu wrote:
> On Sun, Oct 11, 2015 at 11:29 AM Ronald S. Bultje <rsbultje at gmail.com>
> wrote:
> 
>> It's an a non-installed header and only used in one place (flacenc).
>> Since ff_ctz is static inline, it's fine to use that instead.
>> ---
>>  libavcodec/flacenc.c | 2 +-
>>  libavutil/intmath.c  | 5 -----
>>  libavutil/intmath.h  | 8 --------
>>  3 files changed, 1 insertion(+), 14 deletions(-)
>>
> 
> Should we change ff_ctz to avpriv_ctz? Using a ff_ from another library
> just doesn't look right. (Or we could remove the prefix like emms_c but
> ctz() sounds too generic)
> 
> Timothy

ff_ctz is an inline function. The name is irrelevant. Just leave it as is.



More information about the ffmpeg-devel mailing list