[Ffmpeg-devel] Re: [PATCH] QTRLE encoder

Aurelien Jacobs aurel
Thu Feb 15 01:32:54 CET 2007


On Thu, 15 Feb 2007 01:23:20 +0100
Baptiste Coudurier <baptiste.coudurier at smartjog.com> wrote:

> Hi
> 
> Clemens Fruhwirth wrote:
> > Thank you Michael for your very detailed reply. I read all your
> > suggestions and I agree with all of them.
> > 
> > I only infinitesimally favour to see this patch merged in contrast to
> > an unmerged patch. That's why I won't allocate a personal time slice
> > to play the merging game.
> > 
> > Thank you nonetheless as I'm sure this will be a great help for
> > anybody who has this objective.
> 
> Here is an updated patch. Issues should have been adressed, Michael if
> you see more simplification/optimization, I'll do it. I'll add
> regression tests also, if wanted.
> 
> Index: libavcodec/allcodecs.c
> ===================================================================
> --- libavcodec/allcodecs.c	(revision 7983)
> +++ libavcodec/allcodecs.c	(working copy)
> @@ -109,7 +109,7 @@
>      REGISTER_ENCODER(PPM, ppm);
>      REGISTER_DECODER(QDRAW, qdraw);
>      REGISTER_DECODER(QPEG, qpeg);
> -    REGISTER_DECODER(QTRLE, qtrle);
> +    REGISTER_ENCDEC(QTRLE, qtrle);

Please keep parenthesis indentation when you commit this.

Aurel




More information about the ffmpeg-devel mailing list