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

Justin Ruggles justin.ruggles
Mon Sep 1 19:22:18 CEST 2008


Diego Biurrun wrote:
> 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)

thanks. fixed.

-Justin





More information about the ffmpeg-devel mailing list