[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