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

Reply via email to