[FFmpeg-soc] [soc]: r339 - in eac3: ac3.c ac3.h ac3tab.c ac3tab.h eac3.h eac3_parser.c eac3dec.c ffmpeg.patch

[Diego Biurrun]

On Sat, Jul 07, 2007 at 04:04:27PM +0200, bwolowiec wrote:
> 
> Log:
> This is main code of the E-AC3 decoder. It plays available from hd-dvd samples (http://www1.mplayerhq.hu/MPlayer/samples/evob/). Few extensions are lacking: AHT, Channel coupling, spectral extension and few smaller parts of code (because there are no samples).

Please keep log message lines short.

> --- (empty file)
> +++ eac3/ffmpeg.patch	Sat Jul  7 16:04:27 2007
> +--- libavcodec/allcodecs.c	(wersja 9211)
> ++++ libavcodec/allcodecs.c	(kopia robocza)
> +@@ -260,6 +261,7 @@
> +     /* parsers */
> +     REGISTER_PARSER (AAC, aac);
> +     REGISTER_PARSER (AC3, ac3);
> ++    REGISTER_PARSER (EAC3, eac3);
> +     REGISTER_PARSER (CAVSVIDEO, cavsvideo);
> +     REGISTER_PARSER (DCA, dca);
> +     REGISTER_PARSER (DVBSUB, dvbsub);

not alphabetical order

> +Index: libavcodec/avcodec.h
> +===================================================================
> +--- libavcodec/avcodec.h	(wersja 9211)
> ++++ libavcodec/avcodec.h	(kopia robocza)
> +@@ -222,6 +222,7 @@
> +     CODEC_ID_MPEG4AAC,
> + #endif
> +     CODEC_ID_AC3,
> ++    CODEC_ID_EAC3,
> +     CODEC_ID_DTS,
> +     CODEC_ID_VORBIS,
> +     CODEC_ID_DVAUDIO,

Shouldn't these be added to the end of the list?

> +--- libavformat/allformats.c	(wersja 9211)
> ++++ libavformat/allformats.c	(kopia robocza)
> +@@ -72,6 +72,7 @@
> +     REGISTER_DEMUXER (DV1394, dv1394);
> +     REGISTER_DEMUXER (DXA, dxa);
> +     REGISTER_DEMUXER (EA, ea);
> ++    REGISTER_DEMUXER (AC3, ac3);
> +     REGISTER_MUXDEMUX(FFM, ffm);
> +     REGISTER_MUXDEMUX(FLAC, flac);
> +     REGISTER_DEMUXER (FLIC, flic);

Alphabetical order and isn't it eac3?

Diego