On Sun, 9 Aug 2015, Martin Storsjö wrote:

On Sat, 8 Aug 2015, Andreas Cadhalpun wrote:

On 08.08.2015 18:40, Martin Storsjö wrote:
As Hendrik said, public data symbols are problematic from a portability point of view (they behave differently than public functions in DLLs and require jumping through extra hoops, which we've chosen not to jump through - the MSVC portability is quite
tricky enough as it is without bothering with this aspect).

Thanks for the explanation. (It would have been good to mention this in APIchanges
and/or the commit message adding FF_API_AV_REVERSE.)

The correct solution is to drop this and do the same thing as for e.g. log2_tab.o, which is duplicated in all libs that use it (for this table, it'd be libavcodec and
libavutil only).

Attached patch does that.

Thanks, this does look good to me (although I haven't actually tried it).

Actually, this whole patch is not needed in libav at all. libav does not use av_reverse anywhere, so it can be dropped without problem - the deprecation process for this table seems to have been done correctly, so
dropping it at the next major bump should be just fine.

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to