[MPlayer-cvslog] CVS: main/libvo vo_gl.c, 1.89, 1.90 vo_gl2.c, 1.73, 1.74 w32_common.c, 1.13, 1.14

[Joey Parrish]

On Sun, Aug 21, 2005 at 11:59:48AM +0200, Reimar DXffinger CVS wrote:
> CVS change done by Reimar Döffinger CVS
> 
> Update of /cvsroot/mplayer/main/libvo
> In directory mail:/var2/tmp/cvs-serv17385
> 
> Modified Files:
> 	vo_gl.c vo_gl2.c w32_common.c 
> Log Message:
> -wid support for windows. Not well tested, might still behave a bit weird.

> Index: w32_common.c
> ===================================================================
> RCS file: /cvsroot/mplayer/main/libvo/w32_common.c,v
> retrieving revision 1.13
> retrieving revision 1.14
> diff -u -r1.13 -r1.14
> --- w32_common.c	14 Aug 2005 23:16:17 -0000	1.13
> +++ w32_common.c	21 Aug 2005 09:59:45 -0000	1.14
> @@ -259,6 +263,7 @@
>      resetMode();
>      ShowCursor(1);
>      vo_depthonscreen = 0;
> +    if (WinID >= 0)
>      DestroyWindow(vo_window);
>      vo_window = 0;
>      UnregisterClass(classname, 0);

This last chunk is incorrect.  If WinID >= 0, then the window is
supplied by caller, and we should never destroy it.  Change to
(WinID < 0) please.  Everything else looks correct to me.

--Joey

-- 
"I know Kung Fu." --Darth Vader