[vlc-devel] commit: Qt4: hide button release from video widget too ( Rémi Denis-Courmont )

[git version control]

vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Sun Aug 16 21:17:07 2009 +0300| [818dbe7a023f377c6340925ab17a6bb9489cbddc] | committer: Rémi Denis-Courmont 

Qt4: hide button release from video widget too

It does not seem to matter, but better safe than sorry...

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=818dbe7a023f377c6340925ab17a6bb9489cbddc
---

 modules/gui/qt4/components/interface_widgets.cpp |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/modules/gui/qt4/components/interface_widgets.cpp b/modules/gui/qt4/components/interface_widgets.cpp
index cb0f20f..05f8e2d 100644
--- a/modules/gui/qt4/components/interface_widgets.cpp
+++ b/modules/gui/qt4/components/interface_widgets.cpp
@@ -151,15 +151,16 @@ WId VideoWidget::request( int *pi_x, int *pi_y,
     layout->addWidget( reparentable );
 
 #ifdef Q_WS_X11
-    /* HACK: Only one X11 client can subscribe to mouse click events.
+    /* HACK: Only one X11 client can subscribe to mouse button press events.
      * VLC currently handles those in the video display.
-     * Force Qt4 to unsubscribe from them. */
+     * Force Qt4 to unsubscribe from mouse press and release events. */
     Display *dpy = QX11Info::display();
     Window w = stable->winId();
     XWindowAttributes attr;
 
     XGetWindowAttributes( dpy, w, &attr );
-    XSelectInput( dpy, w, attr.your_event_mask & ~ButtonPressMask );
+    attr.your_event_mask &= ~(ButtonPressMask|ButtonReleaseMask);
+    XSelectInput( dpy, w, attr.your_event_mask );
 #endif
     videoSync();
 #ifndef NDEBUG