[FFmpeg-devel] [PATCH] intmath: remove av_ctz.
jamrial at gmail.com
Tue Oct 13 04:41:41 CEST 2015
On 10/12/2015 3:15 PM, Andreas Cadhalpun wrote:
> On 11.10.2015 23:37, James Almer wrote:
>> On 10/11/2015 6:26 PM, Andreas Cadhalpun wrote:
>>> Anyway, please also remove the entry for av_ctz from APIchanges.
>>> It's not public API, so it never belonged there.
>> Huh, even an APIChanges line? Wonder if intmath.h was public at some
>> point, or if whoever added this wasn't aware it was an internal header.
> I guess it's slightly embarrassing, but this is unfortunately not the
> only exported symbol, which is not declared in an installed header.
> It seems several headers are intended to be installed, but actually aren't:
These two definitely were meant to be installed. WTF.
> On the other hand apparently nobody missed these headers...
> Maybe these functions don't need to be public?
They are meant to be public crypto modules, like aes, sha, etc.
> Then some avpriv functions don't need to be exported:
> avpriv_ac3_parse_header (replaced by avpriv_ac3_parse_header2)
> avpriv_color_frame (only used in libavcodec)
> avpriv_mpa_decode_header2 (libavformat/mp3dec.c uses avpriv_mpa_decode_header)
> avpriv_put_string (only used in libavcodec)
Assuming none of these really need to be exported, we can still rename
them to ff_ prefix if we do it asap.
> avpriv_float_dsp_init (only used in libavutil/float_dsp.c)
This one was used until a year or so ago when avpriv_float_dsp_alloc()
was introduced. I guess it could be renamed with an ff_ prefix (or
merged into the new function, much like its fixed counterpart), since
the new function is preferred to allocate and init the context.
> Finally several seemingly internal symbols are needlessly exported:
These are either used by ffprobe, or "other applications" as michaelni
mentioned when i tried to make them fully internal last year.
> Or did I miss something?
> Best regards,
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
More information about the ffmpeg-devel