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

[Justin Ruggles]

Ronald S. Bultje wrote:
> Hi Justin,
> 
> On Sun, Aug 31, 2008 at 10:24 PM, Justin Ruggles
> <justin.ruggles at gmail.com> wrote:
>> Here is a patch to make a separate CODEC_ID_EAC3.  It adds a separate
>> demuxer, muxer, parser, and decoder for E-AC-3.
>>
>> After this is done, the various other muxers/demuxers can be modified to
>> treat E-AC-3 differently if needed.
> 
> Index: libavcodec/ac3_parser.c
> ===================================================================
> --- libavcodec/ac3_parser.c	(revision 15137)
> +++ libavcodec/ac3_parser.c	(working copy)
> @@ -194,3 +194,11 @@
>      ff_aac_ac3_parse,
>      ff_parse_close,
>  };
> +
> +AVCodecParser eac3_parser = {
> +    { CODEC_ID_EAC3 },
> +    sizeof(AACAC3ParseContext),
> +    ac3_parse_init,
> +    ff_aac_ac3_parse,
> +    ff_parse_close,
> +};
> 
> Can't you just add EAC3 in the array of ac3_parser? MP2/3 does that
> too, I think. No need for the separate entry.
> 
> [...]

new patch attached which shares the parser.

-Justin



-------------- next part --------------
A non-text attachment was scrubbed...
Name: eac3_separate_codec_id.diff
Type: text/x-patch
Size: 6616 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080831/d59c04e5/attachment.bin>