[vlc-commits] Move --x11-display to the XCB window plugin

Rémi Denis-Courmont git at videolan.org
Tue Jan 18 18:05:51 CET 2011


vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Tue Jan 18 18:57:21 2011 +0200| [3d7d74852cc62062b9ec296e4341ab12b14d4676] | committer: Rémi Denis-Courmont

Move --x11-display to the XCB window plugin

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

 modules/video_output/xcb/window.c |   12 +++++++++---
 src/libvlc-module.c               |    9 ---------
 2 files changed, 9 insertions(+), 12 deletions(-)

diff --git a/modules/video_output/xcb/window.c b/modules/video_output/xcb/window.c
index 2c5b248..0fdd2bc 100644
--- a/modules/video_output/xcb/window.c
+++ b/modules/video_output/xcb/window.c
@@ -40,10 +40,15 @@ typedef xcb_atom_t Atom;
 
 #include "xcb_vlc.h"
 
-#define XID_TEXT N_("ID of the video output X window")
+#define DISPLAY_TEXT N_("X11 display")
+#define DISPLAY_LONGTEXT N_( \
+    "Video will be rendered with this X11 display. " \
+    "If empty, the default display will be used.")
+
+#define XID_TEXT N_("X11 window ID")
 #define XID_LONGTEXT N_( \
-    "VLC can embed its video output in an existing X11 window. " \
-    "This is the X identifier of that window (0 means none).")
+    "Video will be embedded in this pre-existing X11 window. " \
+    "If zero, a new window will be created.")
 
 static int  Open (vout_window_t *, const vout_window_cfg_t *);
 static void Close (vout_window_t *);
@@ -76,6 +81,7 @@ vlc_module_begin ()
     set_callbacks (EmOpen, EmClose)
     add_shortcut ("embed-xid")
 
+    add_string ("x11-display", NULL, DISPLAY_TEXT, DISPLAY_LONGTEXT, true)
     add_integer ("drawable-xid", 0, XID_TEXT, XID_LONGTEXT, true)
         change_volatile ()
 
diff --git a/src/libvlc-module.c b/src/libvlc-module.c
index 6701889..b892d0e 100644
--- a/src/libvlc-module.c
+++ b/src/libvlc-module.c
@@ -431,11 +431,6 @@ static const char *const ppsz_align_descriptions[] =
 #define EMBEDDED_LONGTEXT N_( \
     "Embed the video output in the main interface." )
 
-#define DISPLAY_TEXT N_("X11 display")
-#define DISPLAY_LONGTEXT N_( \
-    "X11 hardware display to use. By default VLC will " \
-    "use the value of the DISPLAY environment variable.")
-
 #define FULLSCREEN_TEXT N_("Fullscreen video output")
 #define FULLSCREEN_LONGTEXT N_( \
     "Start video in fullscreen mode" )
@@ -1666,10 +1661,6 @@ vlc_module_begin ()
 #ifdef __APPLE__
        add_deprecated_alias( "macosx-embedded" ) /*deprecated since 0.9.0 */
 #endif
-    add_string( "x11-display", NULL,
-                DISPLAY_TEXT, DISPLAY_LONGTEXT, true )
-        add_deprecated_alias( "xvideo-display" ) /* deprecated since 1.1.0 */
-        //add_deprecated_alias( "glx-display" ) cannot have more than one
     add_bool( "xlib", true, "", "", true )
         change_private ()
     add_bool( "drop-late-frames", 1, DROP_LATE_FRAMES_TEXT,



More information about the vlc-commits mailing list