[vlc-commits] xcb/xvideo: remove XVideo module

Rémi Denis-Courmont git at videolan.org
Sat Feb 23 16:51:09 CET 2019


vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Sat Feb 23 16:53:10 2019 +0200| [7ce46c2ff880bbf3cd26ae0a0d71f2461dfab291] | committer: Rémi Denis-Courmont

xcb/xvideo: remove XVideo module

Originally, XVideo was the HAL for video ("2D") overlay.
Nowadays, XVideo is but a backward compatibility wrapper around the 3D
hardware for use by legacy applications (notably GLAMOR provides XVideo
using OpenGL). VLC supports OpenGL well nowadays, so there is no point
in using XVideo.

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

 configure.ac                      |   2 -
 modules/MODULES_LIST              |   1 -
 modules/video_output/Makefile.am  |   9 -
 modules/video_output/xcb/xvideo.c | 699 --------------------------------------
 po/POTFILES.in                    |   1 -
 5 files changed, 712 deletions(-)

diff --git a/configure.ac b/configure.ac
index 88432841a1..4be9420a14 100644
--- a/configure.ac
+++ b/configure.ac
@@ -3182,8 +3182,6 @@ AS_IF([test "${enable_xcb}" != "no"], [
     xcb_err="$xcb_err ${XCB_SHM_PKG_ERRORS}."])
   PKG_CHECK_MODULES([XCB_XKB], [xcb-xkb],, [
     xcb_err="$xcb_err ${XCB_XKB_PKG_ERRORS}."])
-  PKG_CHECK_MODULES([XCB_XV], [xcb-xv >= 1.1.90.1],, [
-    xcb_err="$xcb_err ${XCB_XV_PKG_ERRORS}."])
 
   dnl xproto
   PKG_CHECK_MODULES(XPROTO, [xproto],, [
diff --git a/modules/MODULES_LIST b/modules/MODULES_LIST
index 20f84cd1c5..7ec47e71d1 100644
--- a/modules/MODULES_LIST
+++ b/modules/MODULES_LIST
@@ -484,7 +484,6 @@ $Id$
  * xcb_screen: input module that takes screenshots of the primary monitor
  * xcb_window: XCB window provider
  * xcb_x11: a X11 video output using XCB
- * xcb_xv: a XVideo video output using XCB
  * xdg_screensaver: xdg-utils screensaver inhibition
  * xdg_shell: XDG shell surface window provider using Wayland-client
  * xml: LibXML xml parser
diff --git a/modules/video_output/Makefile.am b/modules/video_output/Makefile.am
index 6c95e621ca..84183f3f84 100644
--- a/modules/video_output/Makefile.am
+++ b/modules/video_output/Makefile.am
@@ -199,14 +199,6 @@ libxcb_render_plugin_la_CFLAGS = $(AM_CFLAGS) \
 libxcb_render_plugin_la_LIBADD = libvlc_xcb_events.la \
 	$(XCB_LIBS) $(XCB_SHM_LIBS) $(XCB_RENDER_LIBS) $(LIBM)
 
-libxcb_xv_plugin_la_SOURCES = \
-	video_output/xcb/pictures.c video_output/xcb/pictures.h \
-	video_output/xcb/xvideo.c
-libxcb_xv_plugin_la_CFLAGS = $(AM_CFLAGS) \
-	$(XCB_CFLAGS) $(XCB_SHM_CFLAGS) $(XCB_XV_CFLAGS)
-libxcb_xv_plugin_la_LIBADD = libvlc_xcb_events.la \
-	$(XCB_LIBS) $(XCB_SHM_LIBS) $(XCB_XV_LIBS)
-
 libxcb_window_plugin_la_SOURCES = video_output/xcb/window.c
 libxcb_window_plugin_la_CFLAGS = $(AM_CFLAGS) \
 	$(CFLAGS_xcb_window) \
@@ -233,7 +225,6 @@ libxcb_window_plugin_la_SOURCES += \
 	video_output/xcb/vlc_xkb.h video_output/xcb/xkb.c
 libxcb_window_plugin_la_CFLAGS += -DHAVE_XKBCOMMON
 endif
-vout_LTLIBRARIES += libxcb_xv_plugin.la
 if HAVE_EGL
 vout_LTLIBRARIES += libegl_x11_plugin.la
 endif
diff --git a/modules/video_output/xcb/xvideo.c b/modules/video_output/xcb/xvideo.c
deleted file mode 100644
index afc7a69587..0000000000
--- a/modules/video_output/xcb/xvideo.c
+++ /dev/null
@@ -1,699 +0,0 @@
-/**
- * @file xvideo.c
- * @brief X C Bindings video output module for VLC media player
- */
-/*****************************************************************************
- * Copyright © 2009 Rémi Denis-Courmont
- *
- * This program is free software; you can redistribute it and/or modify it
- * under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation; either version 2.1 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
- *****************************************************************************/
-
-#ifdef HAVE_CONFIG_H
-# include <config.h>
-#endif
-
-#include <stdlib.h>
-#include <limits.h>
-#include <assert.h>
-
-#include <xcb/xcb.h>
-#include <xcb/shm.h>
-#include <xcb/xv.h>
-
-#include <vlc_common.h>
-#include <vlc_plugin.h>
-#include <vlc_vout_display.h>
-#include <vlc_picture_pool.h>
-#include <vlc_dialog.h>
-
-#include "pictures.h"
-#include "events.h"
-
-#define ADAPTOR_TEXT N_("XVideo adaptor number")
-#define ADAPTOR_LONGTEXT N_( \
-    "XVideo hardware adaptor to use. By default, VLC will " \
-    "use the first functional adaptor.")
-
-#define FORMAT_TEXT N_("XVideo format id")
-#define FORMAT_LONGTEXT N_( \
-    "XVideo image format id to use. By default, VLC will " \
-    "try to use the best match for the video being played.")
-
-static int Open(vout_display_t *vd, const vout_display_cfg_t *cfg,
-                video_format_t *fmtp, vlc_video_context *context);
-static void Close(vout_display_t *vd);
-static int EnumAdaptors(const char *, int64_t **, char ***);
-
-/*
- * Module descriptor
- */
-vlc_module_begin ()
-    set_shortname (N_("XVideo"))
-    set_description (N_("XVideo output (XCB)"))
-    set_category (CAT_VIDEO)
-    set_subcategory (SUBCAT_VIDEO_VOUT)
-    set_capability ("vout display", 150)
-    set_callbacks (Open, Close)
-
-    add_integer ("xvideo-adaptor", -1,
-                 ADAPTOR_TEXT, ADAPTOR_LONGTEXT, true)
-        change_integer_cb (EnumAdaptors)
-    add_integer ("xvideo-format-id", 0,
-                 FORMAT_TEXT, FORMAT_LONGTEXT, true)
-    add_obsolete_bool ("xvideo-shm") /* removed in 2.0.0 */
-    add_shortcut ("xcb-xv", "xv", "xvideo")
-vlc_module_end ()
-
-#define MAX_PICTURES (128)
-
-struct vout_display_sys_t
-{
-    xcb_connection_t *conn;
-
-    xcb_window_t window; /* drawable X window */
-    xcb_gcontext_t gc;   /* context to put images */
-    xcb_xv_port_t port;  /* XVideo port */
-    uint32_t id;         /* XVideo format */
-    uint32_t data_size;  /* picture byte size (for non-SHM) */
-    bool     swap_uv;    /* U/V pointer must be swapped in a picture */
-    bool shm;            /* whether to use MIT-SHM */
-
-    xcb_xv_query_image_attributes_reply_t *att;
-    picture_pool_t *pool; /* picture pool */
-    vout_display_place_t place;
-};
-
-static picture_pool_t *Pool (vout_display_t *, unsigned);
-static void Display (vout_display_t *, picture_t *);
-static int Control (vout_display_t *, int, va_list);
-
-/**
- * Check that the X server supports the XVideo extension.
- */
-static bool CheckXVideo (vout_display_t *vd, xcb_connection_t *conn)
-{
-    xcb_xv_query_extension_reply_t *r;
-    xcb_xv_query_extension_cookie_t ck = xcb_xv_query_extension (conn);
-    bool ok = false;
-
-    /* We need XVideo 2.2 for PutImage */
-    r = xcb_xv_query_extension_reply (conn, ck, NULL);
-    if (r == NULL)
-        msg_Dbg (vd, "XVideo extension not available");
-    else
-    if (r->major != 2)
-        msg_Dbg (vd, "XVideo extension v%"PRIu8".%"PRIu8" unknown",
-                 r->major, r->minor);
-    else
-    if (r->minor < 2)
-        msg_Dbg (vd, "XVideo extension v%"PRIu8".%"PRIu8" too old",
-                 r->major, r->minor);
-    else
-    {
-        msg_Dbg (vd, "using XVideo extension v%"PRIu8".%"PRIu8,
-                 r->major, r->minor);
-        ok = true;
-    }
-    free (r);
-    return ok;
-}
-
-static vlc_fourcc_t ParseFormat (vlc_object_t *obj,
-                                 const xcb_xv_image_format_info_t *restrict f)
-{
-    if (f->type != XCB_XV_IMAGE_FORMAT_INFO_TYPE_YUV)
-        return 0; /* Ignore RGB, RENDER handles them better. */
-
-    if (f->u_sample_bits != f->v_sample_bits
-     || f->vhorz_u_period != f->vhorz_v_period
-     || f->vvert_u_period != f->vvert_v_period
-     || f->vhorz_y_period != 1 || f->vvert_y_period != 1)
-        return 0; /* Ignore insane formats with different U and V sampling. */
-
-    if (f->y_sample_bits != f->u_sample_bits)
-        return 0; /* ... or with different luma and chroma depth. */
-
-    if (f->y_sample_bits == 8) {
-        switch (f->num_planes) {
-            case 1:
-                if (f->bpp == 16
-                 && f->vhorz_u_period == 2 && f->vvert_u_period == 1) {
-                    if (!strcmp ((const char *)f->vcomp_order, "YUYV"))
-                        return VLC_CODEC_YUYV;
-                    if (!strcmp ((const char *)f->vcomp_order, "UYVY"))
-                        return VLC_CODEC_UYVY;
-                }
-                break;
-            case 3:
-                if (f->bpp == 12
-                 && f->vhorz_u_period == 2 && f->vvert_u_period == 2) {
-                    if (!strcmp ((const char *)f->vcomp_order, "YVU"))
-                        return VLC_CODEC_YV12;
-                    if (!strcmp ((const char *)f->vcomp_order, "YUV"))
-                        return VLC_CODEC_I420;
-                }
-                break;
-        }
-    }
-
-    msg_Err(obj, "unknown XVideo YUV format %"PRIx32" (%.4s)", f->id, f->guid);
-    msg_Dbg(obj, " %"PRIu8" planes, %"PRIu32" bits/pixel, "
-            "%"PRIu32" bits/sample", f->num_planes, f->bpp, f->y_sample_bits);
-    msg_Dbg(obj, " period: %"PRIu32"x%"PRIu32,
-            f->vhorz_u_period, f->vvert_u_period);
-    msg_Warn(obj, " order: %.32s", f->vcomp_order);
-    return 0;
-}
-
-static bool BetterFormat (vlc_fourcc_t a, const vlc_fourcc_t *tab,
-                          unsigned *rankp)
-{
-    for (unsigned i = 0, max = *rankp; i < max && tab[i] != 0; i++)
-        if (tab[i] == a)
-        {
-            *rankp = i;
-            return true;
-        }
-    return false;
-}
-
-static xcb_xv_query_image_attributes_reply_t *
-FindFormat (vlc_object_t *obj, xcb_connection_t *conn, video_format_t *fmt,
-            const xcb_xv_adaptor_info_t *a, uint32_t *idp)
-{
-    /* Order chromas by preference */
-    vlc_fourcc_t tab[2];
-    const vlc_fourcc_t *chromav = tab;
-
-    vlc_fourcc_t chroma = var_InheritInteger (obj, "xvideo-format-id");
-    if (chroma != 0) /* Forced chroma */
-    {
-        tab[0] = chroma;
-        tab[1] = 0;
-    }
-    else if (vlc_fourcc_IsYUV (fmt->i_chroma)) /* YUV chroma */
-    {
-        chromav = vlc_fourcc_GetYUVFallback (fmt->i_chroma);
-    }
-    else /* RGB chroma */
-        return NULL;
-
-    /* Get available image formats */
-    xcb_xv_list_image_formats_reply_t *list =
-        xcb_xv_list_image_formats_reply (conn,
-            xcb_xv_list_image_formats (conn, a->base_id), NULL);
-    if (list == NULL)
-        return NULL;
-
-    /* Check available XVideo chromas */
-    xcb_xv_query_image_attributes_reply_t *attr = NULL;
-    unsigned rank = UINT_MAX;
-
-    for (const xcb_xv_image_format_info_t *f =
-             xcb_xv_list_image_formats_format (list),
-                                          *f_end =
-             f + xcb_xv_list_image_formats_format_length (list);
-         f < f_end;
-         f++)
-    {
-        chroma = ParseFormat (obj, f);
-        if (chroma == 0)
-            continue;
-
-        if (!BetterFormat (chroma, chromav, &rank))
-            continue;
-
-        xcb_xv_query_image_attributes_reply_t *i;
-        i = xcb_xv_query_image_attributes_reply (conn,
-            xcb_xv_query_image_attributes (conn, a->base_id, f->id,
-                                           fmt->i_visible_width,
-                                           fmt->i_visible_height), NULL);
-        if (i == NULL)
-            continue;
-
-        fmt->i_chroma = chroma;
-        fmt->i_x_offset = 0;
-        fmt->i_y_offset = 0;
-        /* TODO: Check pitches and offsets as in PoolAlloc() to increase
-         * i_width and i_height where possible. */
-        fmt->i_width = i->width;
-        fmt->i_height = i->height;
-
-        *idp = f->id;
-        free (attr);
-        attr = i;
-        if (rank == 0)
-            break; /* shortcut for perfect match */
-    }
-
-    free (list);
-    return attr;
-}
-
-
-/**
- * Probe the X server.
- */
-static int Open (vout_display_t *vd, const vout_display_cfg_t *cfg,
-                 video_format_t *fmtp, vlc_video_context *context)
-{
-    vlc_object_t *obj = VLC_OBJECT(vd);
-    vout_display_sys_t *p_sys;
-
-    {   /* NOTE: Reject hardware surface formats. Blending would break. */
-        const vlc_chroma_description_t *chroma =
-            vlc_fourcc_GetChromaDescription(vd->source.i_chroma);
-        if (chroma != NULL && chroma->plane_count == 0)
-            return VLC_EGENERIC;
-    }
-
-    p_sys = malloc (sizeof (*p_sys));
-    if (p_sys == NULL)
-        return VLC_ENOMEM;
-
-    vd->sys = p_sys;
-
-    /* Connect to X */
-    xcb_connection_t *conn;
-    const xcb_screen_t *screen;
-    if (vlc_xcb_parent_Create(vd, cfg, &conn, &screen) == NULL)
-    {
-        free (p_sys);
-        return VLC_EGENERIC;
-    }
-
-    p_sys->conn = conn;
-    p_sys->att = NULL;
-    p_sys->pool = NULL;
-
-    if (!CheckXVideo (vd, conn))
-    {
-        msg_Warn (vd, "Please enable XVideo 2.2 for faster video display");
-        goto error;
-    }
-
-    p_sys->window = xcb_generate_id (conn);
-
-    /* Cache adaptors infos */
-    xcb_xv_query_adaptors_reply_t *adaptors =
-        xcb_xv_query_adaptors_reply (conn,
-            xcb_xv_query_adaptors (conn, cfg->window->handle.xid), NULL);
-    if (adaptors == NULL)
-        goto error;
-
-    int adaptor_selected = var_InheritInteger (obj, "xvideo-adaptor");
-    int adaptor_current = -1;
-
-    /* */
-    video_format_t fmt;
-
-    p_sys->port = 0;
-    vout_display_PlacePicture(&p_sys->place, &vd->source, cfg);
-
-    xcb_xv_adaptor_info_iterator_t it;
-    const xcb_xv_format_t *f;
-
-    for (it = xcb_xv_query_adaptors_info_iterator (adaptors);
-         it.rem > 0;
-         xcb_xv_adaptor_info_next (&it))
-    {
-        const xcb_xv_adaptor_info_t *a = it.data;
-
-        adaptor_current++;
-        if (adaptor_selected != -1 && adaptor_selected != adaptor_current)
-            continue;
-        if (!(a->type & XCB_XV_TYPE_INPUT_MASK)
-         || !(a->type & XCB_XV_TYPE_IMAGE_MASK))
-            continue;
-
-        /* Look for an image format */
-        video_format_ApplyRotation(&fmt, fmtp);
-        free (p_sys->att);
-        p_sys->att = FindFormat (obj, conn, &fmt, a, &p_sys->id);
-        if (p_sys->att == NULL) /* No acceptable image formats */
-            continue;
-
-        /* Look for an X11 visual, create a window */
-        uint_fast16_t i;
-        f = xcb_xv_adaptor_info_formats(a);
-
-        for (i = 0; i < a->num_formats; i++)
-            if (f[i].depth == screen->root_depth)
-                break; /* this would fail anyway */
-
-        if (i == a->num_formats) {
-            msg_Dbg(vd, "no usable X11 visual");
-            continue;
-        }
-
-        p_sys->port = a->base_id;
-
-        /* Found port - initialize selected format */
-        msg_Dbg(vd, "using adaptor %.*s", (int)a->name_size,
-                xcb_xv_adaptor_info_name(a));
-        msg_Dbg(vd, "using port %"PRIu32, p_sys->port);
-        msg_Dbg(vd, "using image format 0x%"PRIx32, p_sys->id);
-        msg_Dbg(vd, "using X11 visual ID 0x%"PRIx32" (depth: %"PRIu8")",
-                f->visual, f->depth);
-        break;
-    }
-    if (!p_sys->port)
-    {
-        free(adaptors);
-        msg_Err (vd, "no available XVideo adaptor");
-        goto error;
-    }
-
-    uint32_t mask =
-        XCB_CW_BACK_PIXEL |
-        XCB_CW_BORDER_PIXEL |
-        XCB_CW_EVENT_MASK |
-        XCB_CW_COLORMAP;
-    const uint32_t list[] = {
-        /* XCB_CW_BACK_PIXEL */
-        screen->black_pixel,
-        /* XCB_CW_BORDER_PIXEL */
-        screen->black_pixel,
-        /* XCB_CW_EVENT_MASK */
-        0,
-        /* XCB_CW_COLORMAP */
-        screen->default_colormap,
-    };
-    xcb_void_cookie_t c;
-
-    c = xcb_create_window_checked(conn, f->depth, p_sys->window,
-                 cfg->window->handle.xid, p_sys->place.x, p_sys->place.y,
-                 p_sys->place.width, p_sys->place.height, 0,
-                 XCB_WINDOW_CLASS_INPUT_OUTPUT, f->visual, mask, list);
-    free(adaptors);
-
-    if (vlc_xcb_error_Check(vd, conn, "cannot create X11 window", c))
-        goto error;
-
-    xcb_map_window(conn, p_sys->window);
-    msg_Dbg (vd, "using X11 window 0x%08"PRIx32, p_sys->window);
-
-    /* Create graphic context */
-    p_sys->gc = xcb_generate_id (conn);
-    xcb_create_gc (conn, p_sys->gc, p_sys->window, 0, NULL);
-    msg_Dbg (vd, "using X11 graphic context 0x%08"PRIx32, p_sys->gc);
-
-    /* Disable color keying if applicable */
-    {
-        xcb_intern_atom_reply_t *r =
-            xcb_intern_atom_reply (conn,
-                xcb_intern_atom (conn, 1, 21, "XV_AUTOPAINT_COLORKEY"), NULL);
-        if (r != NULL && r->atom != 0)
-            xcb_xv_set_port_attribute(conn, p_sys->port, r->atom, 1);
-        free(r);
-    }
-
-    /* Colour space */
-    fmt.space = COLOR_SPACE_BT601;
-    {
-        xcb_intern_atom_reply_t *r =
-            xcb_intern_atom_reply (conn,
-                xcb_intern_atom (conn, 1, 13, "XV_ITURBT_709"), NULL);
-        if (r != NULL && r->atom != 0)
-        {
-            int_fast32_t value = (vd->source.space == COLOR_SPACE_BT709);
-            xcb_xv_set_port_attribute(conn, p_sys->port, r->atom, value);
-            if (value)
-                fmt.space = COLOR_SPACE_BT709;
-        }
-        free(r);
-    }
-
-    p_sys->shm = XCB_shm_Check (obj, conn);
-
-    /* Setup vout_display_t once everything is fine */
-    p_sys->swap_uv = vlc_fourcc_AreUVPlanesSwapped (fmt.i_chroma,
-                                                    fmtp->i_chroma);
-    if (p_sys->swap_uv)
-        fmt.i_chroma = fmtp->i_chroma;
-    *fmtp = fmt;
-
-    vd->pool = Pool;
-    vd->prepare = NULL;
-    vd->display = Display;
-    vd->control = Control;
-
-    (void) context;
-    return VLC_SUCCESS;
-
-error:
-    Close (vd);
-    return VLC_EGENERIC;
-}
-
-
-/**
- * Disconnect from the X server.
- */
-static void Close (vout_display_t *vd)
-{
-    vout_display_sys_t *p_sys = vd->sys;
-
-    if (p_sys->pool)
-        picture_pool_Release (p_sys->pool);
-
-    free (p_sys->att);
-    xcb_disconnect (p_sys->conn);
-    free (p_sys);
-}
-
-static void PoolAlloc (vout_display_t *vd, unsigned requested_count)
-{
-    vout_display_sys_t *p_sys = vd->sys;
-
-    const uint32_t *pitches= xcb_xv_query_image_attributes_pitches (p_sys->att);
-    const uint32_t *offsets= xcb_xv_query_image_attributes_offsets (p_sys->att);
-    const unsigned num_planes= __MIN(p_sys->att->num_planes, PICTURE_PLANE_MAX);
-    p_sys->data_size = p_sys->att->data_size;
-
-    picture_resource_t res = { NULL };
-    for (unsigned i = 0; i < num_planes; i++)
-    {
-        uint32_t data_size;
-        data_size = (i < num_planes - 1) ? offsets[i+1] : p_sys->data_size;
-
-        res.p[i].i_lines = (data_size - offsets[i]) / pitches[i];
-        res.p[i].i_pitch = pitches[i];
-    }
-
-    picture_t *pic_array[MAX_PICTURES];
-    requested_count = __MIN(requested_count, MAX_PICTURES);
-
-    unsigned count;
-    for (count = 0; count < requested_count; count++)
-    {
-        xcb_shm_seg_t seg = p_sys->shm ? xcb_generate_id (p_sys->conn) : 0;
-
-        if (XCB_picture_Alloc (vd, &res, p_sys->data_size, p_sys->conn, seg))
-            break;
-
-        /* Allocate further planes as specified by XVideo */
-        /* We assume that offsets[0] is zero */
-        for (unsigned i = 1; i < num_planes; i++)
-            res.p[i].p_pixels = res.p[0].p_pixels + offsets[i];
-
-        pic_array[count] = XCB_picture_NewFromResource (&vd->fmt, &res,
-                                                        p_sys->conn);
-        if (unlikely(pic_array[count] == NULL))
-            break;
-
-        if (p_sys->swap_uv)
-            /* YVU: swap U and V planes */
-            picture_SwapUV( pic_array[count] );
-    }
-    xcb_flush (p_sys->conn);
-
-    if (count == 0)
-        return;
-
-    p_sys->pool = picture_pool_New (count, pic_array);
-    if (unlikely(p_sys->pool == NULL))
-        while (count > 0)
-            picture_Release(pic_array[--count]);
-}
-
-/**
- * Return a direct buffer
- */
-static picture_pool_t *Pool (vout_display_t *vd, unsigned requested_count)
-{
-    vout_display_sys_t *p_sys = vd->sys;
-
-    if (!p_sys->pool)
-        PoolAlloc (vd, requested_count);
-
-    return p_sys->pool;
-}
-
-/**
- * Sends an image to the X server.
- */
-static void Display (vout_display_t *vd, picture_t *pic)
-{
-    vout_display_sys_t *p_sys = vd->sys;
-    xcb_shm_seg_t segment = XCB_picture_GetSegment(pic);
-    xcb_void_cookie_t ck;
-    video_format_t fmt;
-
-    vlc_xcb_Manage(vd, p_sys->conn);
-
-    video_format_ApplyRotation(&fmt, &vd->source);
-
-    if (segment)
-        ck = xcb_xv_shm_put_image_checked (p_sys->conn, p_sys->port,
-                              p_sys->window, p_sys->gc, segment, p_sys->id, 0,
-                   /* Src: */ fmt.i_x_offset, fmt.i_y_offset,
-                              fmt.i_visible_width, fmt.i_visible_height,
-                   /* Dst: */ 0, 0, p_sys->place.width, p_sys->place.height,
-                /* Memory: */ pic->p->i_pitch / pic->p->i_pixel_pitch,
-                              pic->p->i_lines, false);
-    else
-        ck = xcb_xv_put_image_checked (p_sys->conn, p_sys->port, p_sys->window,
-                          p_sys->gc, p_sys->id,
-                          fmt.i_x_offset, fmt.i_y_offset,
-                          fmt.i_visible_width, fmt.i_visible_height,
-                          0, 0, p_sys->place.width, p_sys->place.height,
-                          pic->p->i_pitch / pic->p->i_pixel_pitch,
-                          pic->p->i_lines,
-                          p_sys->data_size, pic->p->p_pixels);
-
-    /* Wait for reply. See x11.c for rationale. */
-    xcb_generic_error_t *e = xcb_request_check (p_sys->conn, ck);
-    if (e != NULL)
-    {
-        msg_Dbg (vd, "%s: X11 error %d", "cannot put image", e->error_code);
-        free (e);
-    }
-}
-
-static int Control (vout_display_t *vd, int query, va_list ap)
-{
-    vout_display_sys_t *p_sys = vd->sys;
-
-    switch (query)
-    {
-    case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
-    case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
-    case VOUT_DISPLAY_CHANGE_ZOOM:
-    case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
-    case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
-    {
-        vout_display_PlacePicture(&p_sys->place, &vd->source,
-                                  va_arg(ap, const vout_display_cfg_t *));
-
-        /* Move the picture within the window */
-        const uint32_t values[] = { p_sys->place.x, p_sys->place.y,
-                                    p_sys->place.width, p_sys->place.height, };
-        xcb_configure_window (p_sys->conn, p_sys->window,
-                              XCB_CONFIG_WINDOW_X | XCB_CONFIG_WINDOW_Y
-                            | XCB_CONFIG_WINDOW_WIDTH | XCB_CONFIG_WINDOW_HEIGHT,
-                              values);
-        xcb_flush (p_sys->conn);
-        return VLC_SUCCESS;
-    }
-
-    case VOUT_DISPLAY_RESET_PICTURES:
-        vlc_assert_unreachable();
-    default:
-        msg_Err (vd, "Unknown request in XCB vout display");
-        return VLC_EGENERIC;
-    }
-}
-
-static int EnumAdaptors(const char *var, int64_t **vp, char ***tp)
-{
-    /* Connect to X */
-    char *display = config_GetPsz("x11-display");
-    xcb_connection_t *conn;
-    int snum;
-
-    conn = xcb_connect (display, &snum);
-    free (display);
-    if (xcb_connection_has_error (conn) /*== NULL*/)
-        return -1;
-
-    /* Find configured screen */
-    const xcb_setup_t *setup = xcb_get_setup (conn);
-    const xcb_screen_t *scr = NULL;
-    for (xcb_screen_iterator_t i = xcb_setup_roots_iterator (setup);
-         i.rem > 0; xcb_screen_next (&i))
-    {
-        if (snum == 0)
-        {
-            scr = i.data;
-            break;
-        }
-        snum--;
-    }
-
-    if (scr == NULL)
-    {
-        xcb_disconnect (conn);
-        return -1;
-    }
-
-    xcb_xv_query_adaptors_reply_t *adaptors =
-        xcb_xv_query_adaptors_reply (conn,
-            xcb_xv_query_adaptors (conn, scr->root), NULL);
-    xcb_disconnect (conn);
-    if (adaptors == NULL)
-        return -1;
-
-    xcb_xv_adaptor_info_iterator_t it;
-    size_t n = 0;
-
-    for (it = xcb_xv_query_adaptors_info_iterator (adaptors);
-         it.rem > 0;
-         xcb_xv_adaptor_info_next (&it))
-    {
-        const xcb_xv_adaptor_info_t *a = it.data;
-
-        if ((a->type & XCB_XV_TYPE_INPUT_MASK)
-         && (a->type & XCB_XV_TYPE_IMAGE_MASK))
-            n++;
-    }
-
-    int64_t *values = xmalloc ((n + 1) * sizeof (*values));
-    char **texts = xmalloc ((n + 1) * sizeof (*texts));
-    *vp = values;
-    *tp = texts;
-
-    *(values++) = -1;
-    *(texts++) = strdup (N_("Auto"));
-
-    for (it = xcb_xv_query_adaptors_info_iterator (adaptors), n = -1;
-         it.rem > 0;
-         xcb_xv_adaptor_info_next (&it))
-    {
-        const xcb_xv_adaptor_info_t *a = it.data;
-
-        n++;
-
-        if (!(a->type & XCB_XV_TYPE_INPUT_MASK)
-         || !(a->type & XCB_XV_TYPE_IMAGE_MASK))
-            continue;
-
-        *(values++) = n;
-        *(texts++) = strndup (xcb_xv_adaptor_info_name (a), a->name_size);
-    }
-    free (adaptors);
-    (void) var;
-    return values - *vp;
-}
diff --git a/po/POTFILES.in b/po/POTFILES.in
index b1ea5b6147..a45e6a1d2e 100644
--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -1242,7 +1242,6 @@ modules/video_output/wayland/xdg-shell.c
 modules/video_output/xcb/render.c
 modules/video_output/xcb/window.c
 modules/video_output/xcb/x11.c
-modules/video_output/xcb/xvideo.c
 modules/video_output/yuv.c
 modules/video_splitter/clone.c
 modules/video_splitter/panoramix.c



More information about the vlc-commits mailing list