[FFmpeg-devel] [libav-devel] [PATCH 2/6] avutil: remove av_reverse deprecation

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Sat Aug 8 21:38:39 CEST 2015


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.

Best regards,
Andreas


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-avutil-add-ff_reverse-as-av_reverse-replacement.patch
Type: text/x-diff
Size: 8089 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150808/9ceca429/attachment.patch>


More information about the ffmpeg-devel mailing list