[Ffmpeg-devel] Re: [PATCH] FFmpeg theora encoding
Michael Niedermayer
michaelni
Tue Jan 23 15:55:59 CET 2007
Hi
On Mon, Jan 22, 2007 at 03:16:30PM +0000, Paul Richards wrote:
> On 21/01/07, Diego Biurrun <diego at biurrun.de> wrote:
> >On Sun, Jan 21, 2007 at 09:53:47PM +0000, Paul Richards wrote:
> >>
> >> These issues have been addressed.
> >
> >> --- libavcodec/allcodecs.c (revision 7612)
> >> +++ libavcodec/allcodecs.c (working copy)
> >> @@ -200,6 +200,9 @@
> >> +#ifdef CONFIG_LIBTHEORA
> >> + REGISTER_ENCODER(LIBTHEORA, libtheora);
> >> +#endif
> >
> >These are no longer necessary, Mans came up with a system for moving
> >dependency detection to configure, you'll have to add a line there, look
> >at what is done for oggvorbis/libvorbis.
> >
> >> --- libavcodec/avcodec.h (revision 7612)
> >> +++ libavcodec/avcodec.h (working copy)
> >> @@ -2160,6 +2160,7 @@
> >> extern AVCodec mjpeg_encoder;
> >> +extern AVCodec libtheora_encoder;
> >> extern AVCodec ljpeg_encoder;
> >
> >Please put these in alphabetical order.
> >
>
> Done.
[...]
[configure, makefile and related stuff i dont maintain]
[...]
> +
> +/* libtheora includes */
> +#include "theora/theora.h"
shouldnt that rather be <theora/theora.h> ?
[...]
> + avc_context->extradata_size = 0;
> + avc_context->extradata = 0;
these arent needed
except these iam fine with the patch
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070123/e106df64/attachment.pgp>
More information about the ffmpeg-devel
mailing list