On 10/11/2015 4:16 PM, Timothy Gu wrote: > On Sun, Oct 11, 2015 at 11:29 AM Ronald S. Bultje <rsbul...@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. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
