[MPlayer-cvslog] r35758 - in trunk/gui: interface.c ui/video.c

[ib]

Author: ib
Date: Wed Jan 16 20:43:56 2013
New Revision: 35758

Log:
Save the correct video window position.

Additionally, remove the pointless assignment of the video window
position in video.c, because that's already performed during the
ConfigureNotify event. (As a result of this pointless assignment, the
new video window position had only be saved if dragged with the left
mouse button inside the window, not if dragged by the title bar.)

Modified:
   trunk/gui/interface.c
   trunk/gui/ui/video.c

Modified: trunk/gui/interface.c
==============================================================================
--- trunk/gui/interface.c	Wed Jan 16 20:38:44 2013	(r35757)
+++ trunk/gui/interface.c	Wed Jan 16 20:43:56 2013	(r35758)
@@ -266,8 +266,8 @@ void guiDone(void)
         if (gui_save_pos) {
             gui_main_pos_x  = guiApp.mainWindow.X;
             gui_main_pos_y  = guiApp.mainWindow.Y;
-            gui_video_pos_x = guiApp.video.x;
-            gui_video_pos_y = guiApp.video.y;
+            gui_video_pos_x = guiApp.videoWindow.X;
+            gui_video_pos_y = guiApp.videoWindow.Y;
         }
 
         ass_enabled       = gtkASS.enabled;

Modified: trunk/gui/ui/video.c
==============================================================================
--- trunk/gui/ui/video.c	Wed Jan 16 20:38:44 2013	(r35757)
+++ trunk/gui/ui/video.c	Wed Jan 16 20:43:56 2013	(r35758)
@@ -88,9 +88,6 @@ void uiVideoMouseHandle( int Button,int 
                    if ( !guiApp.videoWindow.isFullScreen )
                     {
                      wsMoveWindow( &guiApp.videoWindow,True,RX - sx,RY - sy );
-                     guiApp.video.x = guiApp.videoWindow.X;
-                     guiApp.video.y = guiApp.videoWindow.Y;
-                     // NOTE TO MYSELF: dragging the title bar goes unnoticed?
                     }
                    break;
             case wsPMMouseButton: