Mercurial > mplayer.hg

changeset 35664:9162585304d0

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
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.)
author ib
date Wed, 16 Jan 2013 19:43:56 +0000
parents d1f84b219340
children c31b82bd6092
files gui/interface.c gui/ui/video.c
diffstat 2 files changed, 2 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/gui/interface.c	Wed Jan 16 19:38:44 2013 +0000
+++ b/gui/interface.c	Wed Jan 16 19:43:56 2013 +0000
@@ -266,8 +266,8 @@
         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;
--- a/gui/ui/video.c	Wed Jan 16 19:38:44 2013 +0000
+++ b/gui/ui/video.c	Wed Jan 16 19:43:56 2013 +0000
@@ -88,9 +88,6 @@
                    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: