[MPlayer-cvslog] CVS: main/libvo vo_directx.c,1.59,1.60

[Alexander Strasser]

Hi,

Joey Parrish CVS wrote:
> CVS change done by Joey Parrish CVS
> 
> Update of /cvsroot/mplayer/main/libvo
> In directory mail:/var2/tmp/cvs-serv17460/libvo
> 
> Modified Files:
> 	vo_directx.c 
> Log Message:
> Intercept maximize event and go into fullscreen mode.
> 
> 
> Index: vo_directx.c
> ===================================================================
> RCS file: /cvsroot/mplayer/main/libvo/vo_directx.c,v
> retrieving revision 1.59
> retrieving revision 1.60
> diff -u -r1.59 -r1.60
> --- vo_directx.c	10 Nov 2005 22:25:54 -0000	1.59
> +++ vo_directx.c	11 Nov 2005 13:22:01 -0000	1.60
> @@ -914,6 +914,9 @@
>  				case SC_MONITORPOWER:
>                  mp_msg(MSGT_VO, MSGL_V ,"<vo_directx><INFO>killing screensaver\n" );
>                  return 0;                      
> +				case SC_MAXIMIZE:
> +					if (!vo_fs) control(VOCTRL_FULLSCREEN, NULL);
> +                return 0;                      
>  			}
>  			break;
>  		}

  Is it necesarry? I know that some windows users expect this
strange behaviour for movie players, but it makes it impossible
to maximize the window except doing it by hand.
  At least making it configurable so that normal window handling
can take place would IMHO be better.
  Btw IIRC wmp doesn't do it this way, so at least it is not that
uncommon to not switch to fullscreen when maximize is requested.

  Or am i completely misunderstanding something here?

  Alex (beastd)