[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