[FFmpeg-devel] [PATCH] split codec_id for ac-3 and e-ac-3

Diego Biurrun diego
Mon Sep 1 18:45:50 CEST 2008


On Mon, Sep 01, 2008 at 11:24:52AM -0400, Justin Ruggles wrote:
> 
> Aurelien Jacobs wrote:
> > On Sun, 31 Aug 2008 23:29:16 -0400
> > Justin Ruggles <justin.ruggles at gmail.com> wrote:
> > 
> >> new patch attached which shares the parser.
> >>
> >> --- libavformat/raw.c	(revision 15137)
> >> +++ libavformat/raw.c	(working copy)
> >> @@ -487,8 +487,8 @@
> >>  
> >> -#ifdef CONFIG_AC3_DEMUXER
> >> -static int ac3_probe(AVProbeData *p)
> >> +#if (CONFIG_AC3_DEMUXER || CONFIG_EAC3_DEMUXER)
> > 
> > With #if you need to use ENABLE_* instead of CONFIG_*:
> > +#if (ENABLE_AC3_DEMUXER || ENABLE_EAC3_DEMUXER)
> 
> thanks. i'll fix it.

Except that the whole file uses CONFIG_* everywhere, so please use

#if defined(CONFIG_AC3_DEMUXER) || defined(CONFIG_EAC3_DEMUXER)

Diego




More information about the ffmpeg-devel mailing list