[MPlayer-dev-eng] [PATCH] Fix several gcc compiler warnings in loader/dshow

Dominik Mierzejewski dominik at rangers.eu.org
Wed Oct 30 01:00:29 CET 2002


On Tuesday, 29 October 2002, Fabian Franz wrote:
> Hi,
> 
> there were several warnings when mplayer compiled the files in this directory.
[...]

I have already sent a patch almost identical to this and it was refused.
Check the archives. I got exactly the same answer from Arpi as you did.

[...]
> Index: loader/dshow/DS_VideoDecoder.c
> ===================================================================
> RCS file: /cvsroot/mplayer/main/loader/dshow/DS_VideoDecoder.c,v
> retrieving revision 1.23
> diff -u -r1.23 DS_VideoDecoder.c
> --- loader/dshow/DS_VideoDecoder.c	22 Sep 2002 02:33:27 -0000	1.23
> +++ loader/dshow/DS_VideoDecoder.c	29 Oct 2002 21:54:13 -0000
> @@ -71,14 +71,14 @@
>  	    };
>              
>  static ct check[] = {
> -		{16, fccYUY2, &MEDIASUBTYPE_YUY2, CAP_YUY2},
> -		{12, fccIYUV, &MEDIASUBTYPE_IYUV, CAP_IYUV},
> -		{16, fccUYVY, &MEDIASUBTYPE_UYVY, CAP_UYVY},
> -		{12, fccYV12, &MEDIASUBTYPE_YV12, CAP_YV12},
> -		//{16, fccYV12, &MEDIASUBTYPE_YV12, CAP_YV12},
> -		{16, fccYVYU, &MEDIASUBTYPE_YVYU, CAP_YVYU},
> -		{12, fccI420, &MEDIASUBTYPE_I420, CAP_I420},
> -		{9,  fccYVU9, &MEDIASUBTYPE_YVU9, CAP_YVU9},
> +		{16, fccYUY2, (GUID*)&MEDIASUBTYPE_YUY2, CAP_YUY2},
> +		{12, fccIYUV, (GUID*)&MEDIASUBTYPE_IYUV, CAP_IYUV},
> +		{16, fccUYVY, (GUID*)&MEDIASUBTYPE_UYVY, CAP_UYVY},
> +		{12, fccYV12, (GUID*)&MEDIASUBTYPE_YV12, CAP_YV12},
> +		//{16, fccYV12, (GUID*)&MEDIASUBTYPE_YV12, CAP_YV12},
> +		{16, fccYVYU, (GUID*)&MEDIASUBTYPE_YVYU, CAP_YVYU},
> +		{12, fccI420, (GUID*)&MEDIASUBTYPE_I420, CAP_I420},
> +		{9,  fccYVU9, (GUID*)&MEDIASUBTYPE_YVU9, CAP_YVU9},
>  		{0},
>  	    };

Are you sure about the above?

-- 
MPlayer RPMs maintainer: http://www.piorunek.pl/~dominik/linux/pkgs/mplayer/
"The Universe doesn't give you any points for doing things that are easy."
        -- Sheridan to Garibaldi in Babylon 5:"The Geometry of Shadows"



More information about the MPlayer-dev-eng mailing list