[MPlayer-cvslog] r29374 - trunk/libvo/x11_common.c

[Uoti Urpala]

On Fri, 2009-06-19 at 22:21 +0200, attila wrote:
> Author: attila
> Date: Fri Jun 19 22:21:58 2009
> New Revision: 29374
> 
> Log:
> fix missing event on move that breaks xmga window movement
> broken in r28541
> 
> Modified:
>    trunk/libvo/x11_common.c
> 
> Modified: trunk/libvo/x11_common.c
> ==============================================================================
> --- trunk/libvo/x11_common.c	Fri Jun 19 17:29:51 2009	(r29373)
> +++ trunk/libvo/x11_common.c	Fri Jun 19 22:21:58 2009	(r29374)
> @@ -846,8 +846,9 @@ int vo_x11_check_events(Display * mydisp
>                      break;
>                  {
>                      int old_w = vo_dwidth, old_h = vo_dheight;
> +		    int old_x = vo_dx, old_y = vo_dy;
>                      vo_x11_update_geometry();
> -                    if (vo_dwidth != old_w || vo_dheight != old_h)
> +                    if (vo_dwidth != old_w || vo_dheight != old_h || vo_dx != old_x || vo_dy != old_y)
>                          ret |= VO_EVENT_RESIZE;

That's wrong, moving the window should of course not create RESIZE
events. Those incorrect events cause flicker and load in other VOs which
do not need to care about movement but do need to change the window when
it's resized.