[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