[FFmpeg-devel] [PATCH] Coremake support - ffmpeg_nommx.patch (1/1) - ffmpeg-nommx.patch (1/1) - ff-ffttest-includes.patch (1/1)

[Diego Biurrun]

On Thu, Jun 07, 2007 at 09:12:55AM -0400, Ronald S. Bultje wrote:
> 
> In article <20070521171340.GK16391 at MichaelsNB>,
>  Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> > > In file included from fft-test.c:26:
> > > dsputil.h: In function 'copy_block2':
> > > dsputil.h:688: warning: implicit declaration of function 'ST16'
> > > dsputil.h:688: warning: implicit declaration of function 'LD16'
> > > dsputil.h: In function 'copy_block4':
> > > dsputil.h:699: warning: implicit declaration of function 'ST32'
> > > dsputil.h:699: warning: implicit declaration of function 'LD32'
> > > fft-test.c: In function 'frandom':
> > > fft-test.c:129: warning: implicit declaration of function 'random'
> > > fft-test.c: In function 'help':
> > > fft-test.c:161: warning: implicit declaration of function 'exit'
> > > fft-test.c:161: warning: incompatible implicit declaration of built-in 
> > > function 'exit'
> > > fft-test.c: In function 'main':
> > > fft-test.c:198: warning: implicit declaration of function 'atoi'
> > > fft-test.c:249: warning: implicit declaration of function 'memcpy'
> > > fft-test.c:249: warning: incompatible implicit declaration of built-in 
> > > function 'memcpy'
> > > fft-test.c:276: warning: incompatible implicit declaration of built-in 
> > > function 'memcpy'
> > > 
> > > I can work around this by adding includes for stdlib.h and string.h to 
> > > fft-test.c.
> 
> See attached. I also removed the undef exit because it is no longer 
> needed if we don't use HAVE_AV_CONFIG_H.
> 
> --- libavcodec/fft-test.c	(revision 9221)
> +++ libavcodec/fft-test.c	(working copy)
> @@ -27,9 +27,9 @@
>  #include <math.h>
>  #include <unistd.h>
>  #include <sys/time.h>
> +#include <stdlib.h>
> +#include <string.h>
>  
> -#undef exit
> -
>  int mm_flags;

This is wrong, we compile with HAVE_AV_CONFIG_H and need the #undef.

Diego