[MPlayer-dev-eng] squelch some warnings (patches incl.)

[Diego Biurrun]

On Fri, Feb 22, 2008 at 02:56:46PM +0100, Elias Pipping wrote:
> 
> I've attached a couple of patches to squelch some warnings, mostly on
> Mac OS X (tested with 10.5.2 on an Intel Mac).
> 
> --- cpudetect.c
> +++ cpudetect.c
> @@ -20,7 +20,7 @@
>  #include <machine/cpu.h>
>  #endif
>  
> -#if defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__)
> +#if defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__) || defined(__APPLE__)
>  #include <sys/types.h>
>  #include <sys/sysctl.h>
>  #endif

This one is applied.

> --- libao2/ao_macosx.c
> +++ libao2/ao_macosx.c
> @@ -46,6 +46,7 @@
>  #include <inttypes.h>
>  #include <pthread.h>
>  #include <sys/types.h>
> +#include <sys/time.h>
>  #include <unistd.h>

This makes no sense to me.  I did not see a warning in the list of
warnings you posted that might be caused by this.

> --- loader/ldt_keeper.c
> +++ loader/ldt_keeper.c
> @@ -58,6 +58,11 @@
>  #include <machine/sysarch.h>
>  #endif
>  
> +#if defined(__APPLE__)
> +#include <architecture/i386/table.h>
> +#include <i386/user_ldt.h>
> +#endif

Applied.

> --- libmpcodecs/mp_image.h	(revision 26061)
> +++ libmpcodecs/mp_image.h	(working copy)
> @@ -1,5 +1,6 @@
>  #ifndef MPLAYER_MP_IMAGE_H
>  #define MPLAYER_MP_IMAGE_H
> +#include <stdlib.h>

I have a more extensive solution in my local tree since some time, I
will clean it up and post it now.

Diego