[FFmpeg-devel] [PATCH] intmath: remove av_ctz.

Michael Niedermayer michael at niedermayer.cc
Tue Oct 13 04:21:20 CEST 2015


On Mon, Oct 12, 2015 at 08:15:10PM +0200, 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:

> libavutil/des.h:
>  av_des_alloc
>  av_des_crypt
>  av_des_init
>  av_des_mac
> libavutil/rc4.h:
>  av_rc4_alloc
>  av_rc4_crypt
>  av_rc4_init
> libavutil/tree.h
>  av_tree_destroy
>  av_tree_enumerate
>  av_tree_find
>  av_tree_insert
>  av_tree_node_alloc
>  av_tree_node_size

the ones above are IMHO intended to be public

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

What does censorship reveal? It reveals fear. -- Julian Assange
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151013/2a0d117b/attachment.sig>


More information about the ffmpeg-devel mailing list