XCB/XVideo: ignore the overlay setting
authorRémi Denis-Courmont <remi@remlab.net>
Sat, 30 Apr 2016 15:50:55 +0000 (18:50 +0300)
committerRémi Denis-Courmont <remi@remlab.net>
Sat, 30 Apr 2016 15:56:53 +0000 (18:56 +0300)
XVideo version 2 does not provide visiblity or control over the use
of hardware video overlays. That is an implementation detail of the
X11 driver. Thus the overlay setting did not really work as advertized.
The XVideo adapter setting has to be used instead.

In fact most drivers still in use today do not use overlay, but
the 3D engine (either via vendor-specific support or via GLAMOR). And
since the XVideo plugin got lower priority than VDPAU and OpenGL, the
setting has been redundant: it only has effects if neither VDPAU nor
OpenGL are supported (!), and even --no-overlay has the same results
as --vout=x11.

modules/video_output/xcb/xvideo.c

index 83fcbf3..b487297 100644 (file)
@@ -355,9 +355,6 @@ static int Open (vlc_object_t *obj)
     vout_display_t *vd = (vout_display_t *)obj;
     vout_display_sys_t *p_sys;
 
-    if (!var_InheritBool (obj, "overlay"))
-        return VLC_EGENERIC;
-    else
     {   /* NOTE: Reject hardware surface formats. Blending would break. */
         const vlc_chroma_description_t *chroma =
             vlc_fourcc_GetChromaDescription(vd->source.i_chroma);