[FFmpeg-devel] [PATCH 2/2] avutil: Move prototype of av_ctz to common.h

Hendrik Leppkes h.leppkes at gmail.com
Sat Aug 22 20:00:55 CEST 2015


On Sat, Aug 22, 2015 at 7:58 PM, Timothy Gu <timothygu99 at gmail.com> wrote:
> On Sat, Aug 22, 2015 at 10:31 AM Hendrik Leppkes <h.leppkes at gmail.com>
> wrote:
>
>> On Sat, Aug 22, 2015 at 7:15 PM, Timothy Gu <timothygu99 at gmail.com> wrote:
>> > intmath.h is not installed so it cannot be considered "public".
>> >
>> > Instead, handle av_ctz the same way as av_log2.
>> > ---
>> >  libavcodec/flacenc.c |  2 +-
>> >  libavutil/common.h   | 10 ++++++++++
>> >  libavutil/intmath.c  |  1 +
>> >  libavutil/intmath.h  |  8 +-------
>> >  4 files changed, 13 insertions(+), 8 deletions(-)
>>
>> Maybe it was never meant to be public?
>>
>
> Well, it is used in libraries other than libavutil, has the `av_` prefix,
> and has an APIchanges entry…

Well, ff_ctz is a inline function in the header, so it can be used
cross-library without the av_ version. Just sayin'


More information about the ffmpeg-devel mailing list