[FFmpeg-devel] Patch: wrong_run should be protected by CONFIG_ENCODERS

avcoder ffmpeg
Mon Jan 4 13:59:16 CET 2010


On Mon, Jan 4, 2010 at 8:00 PM, Diego Biurrun <diego at biurrun.de> wrote:

> On Mon, Jan 04, 2010 at 11:22:08AM +0800, avcoder wrote:
> >
> > wrong_run is only called in CONFIG_ENCODERS
>
> Maybe, but
>
> > --- h263data.h (revision 21009)
> > +++ h263data.h (working copy)
>
> Create your patches from the root of the source tree.
>
> > @@ -254,6 +254,7 @@
> >      intra_level_aic,
> >  };
> >
> > +#ifdef CONFIG_ENCODERS
>
> This is pointless, CONFIG_ENCODERS is always defined.
>
>
Thanks for your comments

Update version

Index: libavcodec/h263data.h
===================================================================
--- libavcodec/h263data.h (revision 21012)
+++ libavcodec/h263data.h (working copy)
@@ -254,6 +254,7 @@
     intra_level_aic,
 };

+#if CONFIG_ENCODERS
 static const uint8_t wrong_run[102] = {
  1,  2,  3,  5,  4, 10,  9,  8,
 11, 15, 17, 16, 23, 22, 21, 20,
@@ -269,6 +270,7 @@
 11, 12, 13, 14,  1, 21, 20, 18,
 19,  2,  1, 34, 35, 36
 };
+#endif

 static const uint16_t h263_format[8][2] = {
     { 0, 0 },

-- 
-----------------------------------------------------------------------------------------
My key fingerprint: d1:03:f5:32:26:ff:d7:3c:e4:42:e3:51:ec:92:78:b2
Inspired by http://www.nextplayer.net. Your potential. Our passion.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: h263data_v2.diff
Type: application/octet-stream
Size: 527 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100104/0033ce66/attachment.obj>



More information about the ffmpeg-devel mailing list