Commit 9f9f6d11 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

Remove the Xlib no-ARGB hack

The XCB/XVideo (for a long time) and the XCB/X11 (since the commit
before the previous one) video output plugins will fail safe in case
the embedding window has an alpha channel. In VLC 1.0, the Xlib plugins
needed this hack as the Xlib error handler would terminate VLC.

If compositing is in use, VLC will now use GLX. This will require
software chroma conversion from YUV to RGB. Also, unless the GLX output
is explicitly selected, VLC will flash twice before it starts rendering.
First, it tries (and fail) to use XVideo, then plain X11. To fix this
cosmetic problem, we would need to keep the window across plugins.

I assume this change is needed to fix #3581, but I have not tested, so
it may or may not be sufficient.
parent b660648d
......@@ -70,10 +70,6 @@ int main( int i_argc, const char *ppsz_argv[] )
/* Disable the ugly Gnome crash dialog so that we properly segfault */
putenv( (char *)"GNOME_DISABLE_CRASH_DIALOG=1" );
# endif
/* Make Xlib hide visuals with an alphachannel. Ensure that Qt4 will not
* use the alpha channel for the embedded video window. */
putenv( (char *)"XLIB_SKIP_ARGB_VISUALS=1" );
#endif
#ifdef HAVE_SETENV
/* Clear the X.Org startup notification ID. Otherwise the UI might try to
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment