[FFmpeg-devel] [PATCH] intmath: remove av_ctz.
Timothy Gu
timothygu99 at gmail.com
Sun Oct 11 21:16:52 CEST 2015
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
More information about the ffmpeg-devel
mailing list