On Thu, Oct 11, 2012 at 07:56:26PM +0300, Martin Storsjö wrote: > On Thu, 11 Oct 2012, Diego Biurrun wrote: > > >--- > >I have duplicated the table with an ff_ prefix and marked it as to > >be removed on the next libavutil bump. Alternatively, one could > >play tricks with the preprocessor and avoid the source-level > >duplication, but I thought this was not worth the trouble. > > > >--- a/libavutil/mathematics.c > >+++ b/libavutil/mathematics.c > >@@ -26,9 +26,12 @@ > > > >-const uint8_t av_reverse[256]={ > >+#ifndef FF_API_AV_REVERSE > >+const uint8_t ff_reverse[256]={ > > You're renaming the original table here as well, which I guess you > didn't intend to? That'd break anything that tries to link to it > currently.
Crap, that's a remnant from the sed expression, updating ... Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel