[vlc-devel] commit: Patch to enable/disable key and mouse handling at vout level: ( Jean-Baptiste Kempf )

git version control git at videolan.org
Mon Nov 17 16:53:03 CET 2008


vlc | branch: master | Jean-Baptiste Kempf <jb at videolan.org> | Mon Nov 17 16:48:04 2008 +0100| [52f79869de83fe0dbc0c199da89f02147f521578] | committer: Jean-Baptiste Kempf 

Patch to enable/disable key and mouse handling at vout level:

- x11-event option is renamed vout-event and become global
(libvlc-module.c)
- var_CreateGetInteger is called once at x11 initialization

Patch by Joseph Tulou

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

 modules/video_output/x11/x11.c     |   12 ------------
 modules/video_output/x11/xcommon.c |   26 +++++++++++++-------------
 modules/video_output/x11/xcommon.h |    3 +++
 src/libvlc-module.c                |   12 ++++++++++++
 4 files changed, 28 insertions(+), 25 deletions(-)

diff --git a/modules/video_output/x11/x11.c b/modules/video_output/x11/x11.c
index 88522f8..179e3d5 100644
--- a/modules/video_output/x11/x11.c
+++ b/modules/video_output/x11/x11.c
@@ -66,16 +66,6 @@ extern void Deactivate ( vlc_object_t * );
     "Screen to use in fullscreen mode. For instance " \
     "set it to 0 for first screen, 1 for the second.")
 
-#define X11_EVENT_TEXT N_("key and mouse event handling at X11 plugin level.")
-#define X11_EVENT_LONGTEXT N_( \
-    "This parameter accepts values : 1 (full event handling support), " \
-    "2 (event handling only for fullscreen) or 3 (No event handling). "  \
-    "Full event handling support is the default value.")
-
-static const int pi_x11_event_values[] = { 1, 2, 3 };
-static const char *const ppsz_x11_event_descriptions[] =
-     { N_("FullSupport"), N_("Fullscreen-Only"), N_("None") };
-
 vlc_module_begin ()
     set_shortname( "X11" )
     set_category( CAT_VIDEO )
@@ -88,8 +78,6 @@ vlc_module_begin ()
 #ifdef HAVE_XINERAMA
     add_integer ( "x11-xineramascreen", -1, NULL, SCREEN_TEXT, SCREEN_LONGTEXT, true )
 #endif
-    add_integer( "x11-event", 1, NULL, X11_EVENT_TEXT, X11_EVENT_LONGTEXT, true )
-        change_integer_list( pi_x11_event_values, ppsz_x11_event_descriptions, NULL )
     set_description( N_("X11 video output") )
     set_capability( "video output", 70 )
     set_callbacks( Activate, Deactivate )
diff --git a/modules/video_output/x11/xcommon.c b/modules/video_output/x11/xcommon.c
index db13559..83a3b88 100644
--- a/modules/video_output/x11/xcommon.c
+++ b/modules/video_output/x11/xcommon.c
@@ -210,6 +210,9 @@ int Activate ( vlc_object_t *p_this )
 
     vlc_mutex_init( &p_vout->p_sys->lock );
 
+    /* key and mouse event handling */
+    p_vout->p_sys->i_vout_event = var_CreateGetInteger( p_vout, "vout-event" );
+
     /* Open display, using the "display" config variable or the DISPLAY
      * environment variable */
     psz_display = config_GetPsz( p_vout, MODULE_STRING "-display" );
@@ -1529,22 +1532,21 @@ static int ManageVideo( vout_thread_t *p_vout )
                            i_x, i_y, i_width, i_height );
     }
 
-    /* cursor hiding depending on --x11-event option
+    /* cursor hiding depending on --vout-event option
      *      activated if:
      *            value = 1 (Fullsupport) (default value)
      *         or value = 2 (Fullscreen-Only) and condition met
      */
-    int i_x11_event = var_CreateGetInteger( p_vout, "x11-event" );
-    bool b_x11_event = (    ( i_x11_event == 1 )
-                         || ( i_x11_event == 2 && p_vout->b_fullscreen )
-                       );
+    bool b_vout_event = (   ( p_vout->p_sys->i_vout_event == 1 )
+                         || ( p_vout->p_sys->i_vout_event == 2 && p_vout->b_fullscreen )
+                        );
 
     /* Autohide Cursour */
     if( mdate() - p_vout->p_sys->i_time_mouse_last_moved >
         p_vout->p_sys->i_mouse_hide_timeout )
     {
         /* Hide the mouse automatically */
-        if( b_x11_event && p_vout->p_sys->b_mouse_pointer_visible )
+        if( b_vout_event && p_vout->p_sys->b_mouse_pointer_visible )
         {
             ToggleCursor( p_vout );
         }
@@ -1800,17 +1802,15 @@ static int CreateWindow( vout_thread_t *p_vout, x11_window_t *p_win )
         }
     } while( !( b_expose && b_configure_notify && b_map_notify ) );
 
-    /* key and mouse events handling depending on --x11-event option
+    /* key and mouse events handling depending on --vout-event option
      *      activated if:
      *            value = 1 (Fullsupport) (default value)
      *         or value = 2 (Fullscreen-Only) and condition met
      */
-    int i_x11_event = var_CreateGetInteger( p_vout, "x11-event" );
-    bool b_x11_event = (    ( i_x11_event == 1 )
-                         || ( i_x11_event == 2 && p_vout->b_fullscreen )
-                       );
-
-    if ( b_x11_event )
+    bool b_vout_event = (   ( p_vout->p_sys->i_vout_event == 1 )
+                         || ( p_vout->p_sys->i_vout_event == 2 && p_vout->b_fullscreen )
+                        );
+    if ( b_vout_event )
         XSelectInput( p_vout->p_sys->p_display, p_win->base_window,
                       StructureNotifyMask | KeyPressMask |
                       ButtonPressMask | ButtonReleaseMask |
diff --git a/modules/video_output/x11/xcommon.h b/modules/video_output/x11/xcommon.h
index 240cb63..aaf8242 100644
--- a/modules/video_output/x11/xcommon.h
+++ b/modules/video_output/x11/xcommon.h
@@ -222,6 +222,9 @@ struct vout_sys_t
     x11_window_t        original_window;
     x11_window_t        fullscreen_window;
 
+    /* key and mouse event handling */
+    int                 i_vout_event;  /* 1(Fullsupport), 2(FullscreenOnly), 3(none) */
+
     /* X11 generic properties */
     bool          b_altfullscreen;          /* which fullscreen method */
 #ifdef HAVE_SYS_SHM_H
diff --git a/src/libvlc-module.c b/src/libvlc-module.c
index 7ee8827..7c52604 100644
--- a/src/libvlc-module.c
+++ b/src/libvlc-module.c
@@ -548,6 +548,16 @@ static const char *const ppsz_pos_descriptions[] =
     "This avoids flooding the message log with debug output from the " \
     "video output synchronization mechanism.")
 
+#define VOUT_EVENT_TEXT N_("key and mouse event handling at vout level.")
+#define VOUT_EVENT_LONGTEXT N_( \
+    "This parameter accepts values : 1 (full event handling support), " \
+    "2 (event handling only for fullscreen) or 3 (No event handling). "  \
+    "Full event handling support is the default value.")
+
+static const int pi_vout_event_values[] = { 1, 2, 3 };
+static const char *const ppsz_vout_event_descriptions[] =
+     { N_("FullSupport"), N_("Fullscreen-Only"), N_("None") };
+
 /*****************************************************************************
  * Input
  ****************************************************************************/
@@ -1523,6 +1533,8 @@ vlc_module_begin ()
               SKIP_FRAMES_LONGTEXT, true );
     add_bool( "quiet-synchro", 0, NULL, QUIET_SYNCHRO_TEXT,
               QUIET_SYNCHRO_LONGTEXT, true );
+    add_integer( "vout-event", 1, NULL, VOUT_EVENT_TEXT, VOUT_EVENT_LONGTEXT, true )
+        change_integer_list( pi_vout_event_values, ppsz_vout_event_descriptions, NULL )
 #ifndef __APPLE__
     add_bool( "overlay", 1, NULL, OVERLAY_TEXT, OVERLAY_LONGTEXT, false )
 #endif




More information about the vlc-devel mailing list