[FFmpeg-devel] [PATCH] MLP/TrueHD decoder
Diego Biurrun
diego
Fri Oct 12 15:24:06 CEST 2007
On Fri, Oct 12, 2007 at 01:29:46PM +0100, Ian Caulfield wrote:
>
> I've been working for a while on reverse engineering the MLP audio
> stream format, and writing my own decoder. I've finally got to a point
> where I have something ready for review. I'm going to try to get the
> documentation I've produced on the format up on the multimediawiki as
> well.
New decoders are always very much welcome :)
> Comments welcome :)
>
> --- libavcodec/mlp_parser.c (revision 0)
> +++ libavcodec/mlp_parser.c (revision 0)
> @@ -0,0 +1,338 @@
> +/*
> + * MLP parser
> + * Copyright (c) 2007 Ian Caulfield.
nit: This period is useless.
> +/**
> + * @file mlp_parser.c
> + * MLP parser.
ditto
> --- libavcodec/mlp_parser.h (revision 0)
> +++ libavcodec/mlp_parser.h (revision 0)
> @@ -0,0 +1,53 @@
> +/*
> + * MLP parser prototypes
> + * Copyright (c) 2007 Ian Caulfield.
ditto
> +/**
> + * @file mlp_parser.h
> + * MLP parser prototypes.
ditto, same in the other patch
A slightly more useful comment: The header file needs multiple
inclusion guards.
The build system part of the patch is OK.
Diego
More information about the ffmpeg-devel
mailing list