[Ffmpeg-devel] [PATCH] proper fix for AC3 parser linkage

[Aurelien Jacobs]

On Tue, 24 Apr 2007 19:26:45 -0400
Justin Ruggles <justinruggles at bellsouth.net> wrote:

> Aurelien Jacobs wrote:
> > On Sat, 21 Apr 2007 21:01:08 +0200
> > matthieu castet <castet.matthieu at free.fr> wrote:
> > 
> > 
> >>Diego Biurrun wrote:
> >>
> >>>Attached is a proper fix for the link failure when disabling the AC3
> >>>encoder while the AC3 parser is still enabled.
> >>>
> >>>It has the added benefit of removing some extern declarations from
> >>>ac3.h, the diffstat should be quite convincing.
> >>>
> >>> 3 files changed, 2 insertions(+), 13 deletions(-)
> >>>
> >>>I'll commit tomorrow if nobody objects.
> >>>
> >>
> >>Hum, this seems broken : now we have ac3tab.h global array declaration 
> >>in ac3.c and parser.c.
> >>And also now these array are duplicated.
> >>That's why I always found ugly to put data declaration and init in 
> >>header file instead of *.c .
> > 
> > 
> > I agree with this.
> > The attached patch will move all the tables in ac3tab.c so that only
> > ac3tab.o is required by parser.o.
> > Note that to apply this patch you must:
> >   svn mv libavcodec/ac3tab.h libavcodec/ac3tab.c
> >   patch -p0 < ac3tab.diff
> >   svn add libavcodec/ac3tab.h
> > 
> > If this patch is ok, I intend to apply it in several steps.
> > 
> > Aurel
> 
> There hasn't been any other response to this in a few days.  So...as
> AC-3 maintainer, I'm giving my okay for your patch.  If you want to wait
> for Michael to chime in or for Diego to respond that's up to you. :)

I wasn't especially waiting for their response... I was just too busy
last few days to apply this patch ;-)
I now applied it, following your suggestion about `svn cp ac3.h ac3tab.h`.

Aurel