[MPlayer-dev-eng] Another set of compiler warning fixes

Arpi arpi at thot.banki.hu
Sat Jun 22 14:21:01 CEST 2002


Hi,

> Looks like most of my fixes were applied to CVS recently.
> However, few are still missing. Apply at your own discretion.

>  
> +void uninit_audio(sh_audio_t *sh_audio);	/* forward declaration */
> +

it's defined in dec_audio.h (not in .c)

> +++ MPlayer-20020618/libmpcodecs/vd.c	Tue Jun 18 19:51:31 2002
> @@ -23,6 +23,8 @@
>  #include "vd.h"
>  #include "vf.h"
>  
> +#include "dec_video.h"
> +

why does vd.c needs it? it shouldn't!

> +++ MPlayer-20020618/libmpcodecs/mp_image.h	Tue Jun 18 19:51:31 2002
> @@ -1,6 +1,8 @@
>  #ifndef __MP_IMAGE_H
>  #define __MP_IMAGE_H 1
>  
> +#include <string.h>
> +

NEVER include headers from headers
(and get .so to understand this...)

>  
> -typedef unsigned long ulong;
> +/* typedef unsigned long ulong; */ /* already defined in <sys/types.h>, which is included by 
<s
> tdlib.h> */

on every supported platforms?

A'rpi
--
Using GyS-Mailer3 GSM Edition :)





More information about the MPlayer-dev-eng mailing list