[vlc-devel] [PATCH 2/2] sdl: remove video output

RĂ©mi Denis-Courmont remi at remlab.net
Sat Aug 5 20:14:51 CEST 2017


The SDL video output has been forcefully disabled on MacOS X for 12
years. It has also been disabled in Windows builds for over 6 years.
Finally, it has been broken on X11 due to lack of windowing
integration, for several years, cannot be fixed without changes to
the SDL API, and is completely useless.
---
 configure.ac                     |  30 +-
 modules/MODULES_LIST             |   1 -
 modules/video_output/Makefile.am |  15 -
 modules/video_output/sdl.c       | 713 ---------------------------------------
 po/POTFILES.in                   |   1 -
 5 files changed, 8 insertions(+), 752 deletions(-)
 delete mode 100644 modules/video_output/sdl.c

diff --git a/configure.ac b/configure.ac
index 7501bee1ba..8487fea895 100644
--- a/configure.ac
+++ b/configure.ac
@@ -3118,31 +3118,17 @@ AM_CONDITIONAL([HAVE_WAYLAND_EGL], [test "${have_wayland_egl}" = "yes"])
 
 
 dnl
-dnl  SDL module
+dnl  SDL image decoder module
 dnl
-AC_ARG_ENABLE(sdl,
-  [  --enable-sdl            SDL support (default enabled)])
 AC_ARG_ENABLE(sdl-image,
   [  --enable-sdl-image      SDL image support (default enabled)])
-if test "${enable_sdl}" != "no"
-then
-   PKG_CHECK_MODULES(SDL, [sdl >= 1.2.10], [
-      # SDL on Darwin is heavily patched and can only run SDL_image
-      if test "${SYS}" != "darwin"; then
-        VLC_ADD_PLUGIN([vout_sdl])
-      fi
-
-      # SDL_image
-      AS_IF([ test "${enable_sdl_image}" != "no"],[
-        PKG_CHECK_MODULES(SDL_IMAGE, [SDL_image >= 1.2.10], [
-          VLC_ADD_PLUGIN([sdl_image])],
-          [ AC_MSG_WARN([${SDL_IMAGE_PKG_ERRORS}. You should install it alongside your SDL package.])
-          ])
-      ])
-   ],[
-     AC_MSG_WARN([${SDL_PKG_ERRORS}.])
-   ])
-fi
+AS_IF([test "${enable_sdl_image}" != "no"],[
+  PKG_CHECK_MODULES(SDL_IMAGE, [SDL_image >= 1.2.10], [
+    VLC_ADD_PLUGIN([sdl_image])
+  ], [
+    AC_MSG_WARN([${SDL_IMAGE_PKG_ERRORS}. You should install it alongside your SDL package.])
+  ])
+])
 
 dnl
 dnl  freetype module
diff --git a/modules/MODULES_LIST b/modules/MODULES_LIST
index d125a999e0..9509eba71d 100644
--- a/modules/MODULES_LIST
+++ b/modules/MODULES_LIST
@@ -447,7 +447,6 @@ $Id$
  * vorbis: a vorbis audio decoder/packetizer using the libvorbis library
  * vout_ios: iOS video provider using OpenGL ES 2
  * vout_macosx: Mac OS X OpenGL provider
- * vout_sdl: video output module using the SDL library
  * vpx: WebM encoder and decoder (VP8/VP9)
  * vsxu: audio visualization using Vovoid VSXu
  * wall: image wall filter
diff --git a/modules/video_output/Makefile.am b/modules/video_output/Makefile.am
index 6c5ee08057..e25fc40a99 100644
--- a/modules/video_output/Makefile.am
+++ b/modules/video_output/Makefile.am
@@ -405,21 +405,6 @@ EXTRA_LTLIBRARIES += libfb_plugin.la
 vout_LTLIBRARIES += $(LTLIBfb)
 
 
-### Simple DirectMedia Layer ###
-libvout_sdl_plugin_la_SOURCES = video_output/sdl.c
-libvout_sdl_plugin_la_CFLAGS = $(AM_CFLAGS) $(SDL_CFLAGS)
-libvout_sdl_plugin_la_LIBADD = $(SDL_LIBS)
-if !HAVE_WIN32
-if !HAVE_OS2
-libvout_sdl_plugin_la_LIBADD += $(X_LIBS) $(X_PRE_LIBS) -lX11
-endif
-endif
-libvout_sdl_plugin_la_LDFLAGS = $(AM_LDFLAGS) -rpath '$(voutdir)'
-EXTRA_LTLIBRARIES += libvout_sdl_plugin.la
-vout_LTLIBRARIES += $(LTLIBvout_sdl)
-
-
-
 ### ASCII Art ###
 libaa_plugin_la_SOURCES = video_output/aa.c
 libaa_plugin_la_LIBADD = -laa
diff --git a/modules/video_output/sdl.c b/modules/video_output/sdl.c
deleted file mode 100644
index 51c935cbe0..0000000000
--- a/modules/video_output/sdl.c
+++ /dev/null
@@ -1,713 +0,0 @@
-/*****************************************************************************
- * sdl.c: SDL video output display method
- *****************************************************************************
- * Copyright (C) 1998-2009 VLC authors and VideoLAN
- * $Id$
- *
- * Authors: Samuel Hocevar <sam at zoy.org>
- *          Pierre Baillet <oct at zoy.org>
- *          Arnaud de Bossoreille de Ribou <bozo at via.ecp.fr>
- *          Laurent Aimar <fenrir _AT_ videolan _DOT_ org>
- *
- * 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.
- *****************************************************************************/
-
-/*****************************************************************************
- * Preamble
- *****************************************************************************/
-#ifdef HAVE_CONFIG_H
-# include "config.h"
-#endif
-
-#include <vlc_common.h>
-#include <vlc_plugin.h>
-#include <vlc_vout_display.h>
-#include <vlc_picture_pool.h>
-
-#include <assert.h>
-
-#include <SDL.h>
-
-#if !defined(_WIN32) && !defined(__OS2__)
-# ifdef X_DISPLAY_MISSING
-#  error Xlib required due to XInitThreads
-# endif
-# include <vlc_xlib.h>
-#endif
-
-/*****************************************************************************
- * Module descriptor
- *****************************************************************************/
-static int  Open (vlc_object_t *);
-static void Close(vlc_object_t *);
-
-#define CHROMA_TEXT N_("SDL chroma format")
-#define CHROMA_LONGTEXT N_(\
-    "Force the SDL renderer to use a specific chroma format instead of " \
-    "trying to improve performances by using the most efficient one.")
-
-#define OVERLAY_TEXT N_("YUV overlay")
-#define OVERLAY_LONGTEXT N_(\
-    "Use the hardware YUV overlay of the graphic card (if available).")
-
-vlc_module_begin()
-    set_shortname("SDL")
-    set_category(CAT_VIDEO)
-    set_subcategory(SUBCAT_VIDEO_VOUT)
-    set_description(N_("Simple DirectMedia Layer video output"))
-    set_capability("vout display", 70)
-    add_shortcut("sdl")
-    add_bool("sdl-overlay", true, OVERLAY_TEXT, OVERLAY_LONGTEXT, false)
-    add_string("sdl-chroma", NULL, CHROMA_TEXT, CHROMA_LONGTEXT, true)
-    add_obsolete_string("sdl-video-driver") /* obsolete since 1.1.0 */
-    set_callbacks(Open, Close)
-#if defined(__i386__) || defined(__x86_64__)
-    /* On i386, SDL is linked against svgalib */
-    cannot_unload_broken_library()
-#endif
-vlc_module_end()
-
-
-/*****************************************************************************
- * Local prototypes
- *****************************************************************************/
-static picture_pool_t *Pool  (vout_display_t *, unsigned);
-static void           PictureDisplay(vout_display_t *, picture_t *, subpicture_t *);
-static int            Control(vout_display_t *, int, va_list);
-static void           Manage(vout_display_t *);
-
-/* */
-static int ConvertKey(SDLKey);
-
-/* */
-static vlc_mutex_t sdl_lock = VLC_STATIC_MUTEX;
-
-/* */
-struct vout_display_sys_t {
-    vout_display_place_t place;
-
-    SDL_Surface          *display;
-    int                  display_bpp;
-    uint32_t             display_flags;
-
-    unsigned int         desktop_width;
-    unsigned int         desktop_height;
-
-    /* For YUV output */
-    SDL_Overlay          *overlay;
-    bool                 is_uv_swapped;
-
-    /* */
-    picture_pool_t       *pool;
-};
-
-/**
- * This function initializes SDL vout method.
- */
-static int Open(vlc_object_t *object)
-{
-    vout_display_t *vd = (vout_display_t *)object;
-    vout_display_sys_t *sys;
-
-    if (vout_display_IsWindowed(vd))
-        return VLC_EGENERIC;
-#if !defined(_WIN32) && !defined(__OS2__)
-    if (!vlc_xlib_init (object))
-        return VLC_EGENERIC;
-#endif
-
-    /* XXX: check for conflicts with the SDL audio output */
-    vlc_mutex_lock(&sdl_lock);
-
-    /* Check if SDL video module has been initialized */
-    if (SDL_WasInit(SDL_INIT_VIDEO) != 0) {
-        vlc_mutex_unlock(&sdl_lock);
-        return VLC_EGENERIC;
-    }
-
-    vd->sys = sys = calloc(1, sizeof(*sys));
-    if (!sys) {
-        vlc_mutex_unlock(&sdl_lock);
-        return VLC_ENOMEM;
-    }
-
-    /* */
-    int sdl_flags = SDL_INIT_VIDEO;
-#ifndef _WIN32
-    /* Win32 SDL implementation doesn't support SDL_INIT_EVENTTHREAD yet*/
-    sdl_flags |= SDL_INIT_EVENTTHREAD;
-#endif
-    /* In debug mode you may want vlc to dump a core instead of staying stuck */
-    sdl_flags |= SDL_INIT_NOPARACHUTE;
-
-    /* Initialize library */
-    if (SDL_Init(sdl_flags) < 0) {
-        vlc_mutex_unlock(&sdl_lock);
-
-        msg_Err(vd, "cannot initialize SDL (%s)", SDL_GetError());
-        free(sys);
-        return VLC_EGENERIC;
-    }
-    vlc_mutex_unlock(&sdl_lock);
-
-    /* Translate keys into unicode */
-    SDL_EnableUNICODE(1);
-
-    /* Get the desktop resolution */
-    /* FIXME: SDL has a problem with virtual desktop */
-    sys->desktop_width  = SDL_GetVideoInfo()->current_w;
-    sys->desktop_height = SDL_GetVideoInfo()->current_h;
-
-    /* */
-    video_format_t fmt;
-    video_format_ApplyRotation(&fmt, &vd->fmt);
-
-    /* */
-    vout_display_info_t info = vd->info;
-
-    info.needs_event_thread = true;
-
-    /* Set main window's size */
-    int display_width;
-    int display_height;
-    if (vd->cfg->is_fullscreen) {
-        display_width  = sys->desktop_width;
-        display_height = sys->desktop_height;
-    } else {
-        display_width  = vd->cfg->display.width;
-        display_height = vd->cfg->display.height;
-    }
-
-    /* Initialize flags and cursor */
-    sys->display_flags = SDL_ANYFORMAT | SDL_HWPALETTE | SDL_HWSURFACE | SDL_DOUBLEBUF;
-    sys->display_flags |= vd->cfg->is_fullscreen ? SDL_FULLSCREEN : SDL_RESIZABLE;
-
-    sys->display_bpp = SDL_VideoModeOK(display_width, display_height,
-                                       16, sys->display_flags);
-    if (sys->display_bpp == 0) {
-        msg_Err(vd, "no video mode available");
-        goto error;
-    }
-
-    sys->display = SDL_SetVideoMode(display_width, display_height,
-                                    sys->display_bpp, sys->display_flags);
-    if (!sys->display) {
-        msg_Err(vd, "cannot set video mode");
-        goto error;
-    }
-
-    /* We keep the surface locked forever */
-    SDL_LockSurface(sys->display);
-
-    /* */
-    vlc_fourcc_t forced_chroma = 0;
-    char *psz_chroma = var_InheritString(vd, "sdl-chroma");
-    if (psz_chroma) {
-        forced_chroma = vlc_fourcc_GetCodecFromString(VIDEO_ES, psz_chroma);
-        if (forced_chroma)
-            msg_Dbg(vd, "Forcing chroma to 0x%.8x (%4.4s)",
-                    forced_chroma, (const char*)&forced_chroma);
-        free(psz_chroma);
-    }
-
-    /* Try to open an overlay if requested */
-    sys->overlay = NULL;
-    const bool is_overlay = var_InheritBool(vd, "sdl-overlay");
-    if (is_overlay) {
-        static const struct
-        {
-            vlc_fourcc_t vlc;
-            uint32_t     sdl;
-        } vlc_to_sdl[] = {
-            { VLC_CODEC_YV12, SDL_YV12_OVERLAY },
-            { VLC_CODEC_I420, SDL_IYUV_OVERLAY },
-            { VLC_CODEC_YUYV, SDL_YUY2_OVERLAY },
-            { VLC_CODEC_UYVY, SDL_UYVY_OVERLAY },
-            { VLC_CODEC_YVYU, SDL_YVYU_OVERLAY },
-
-            { 0, 0 }
-        };
-        const vlc_fourcc_t forced_chromas[] = {
-            forced_chroma, 0
-        };
-        const vlc_fourcc_t *fallback_chromas =
-            vlc_fourcc_GetYUVFallback(fmt.i_chroma);
-        const vlc_fourcc_t *chromas = forced_chroma ? forced_chromas : fallback_chromas;
-
-        for (int pass = forced_chroma ? 1 : 0; pass < 2 && !sys->overlay; pass++) {
-            for (int i = 0; chromas[i] != 0; i++) {
-                const vlc_fourcc_t vlc = chromas[i];
-
-                uint32_t sdl = 0;
-                for (int j = 0; vlc_to_sdl[j].vlc != 0 && !sdl; j++) {
-                    if (vlc_to_sdl[j].vlc == vlc)
-                        sdl = vlc_to_sdl[j].sdl;
-                }
-                if (!sdl)
-                    continue;
-
-                sys->overlay = SDL_CreateYUVOverlay(fmt.i_width, fmt.i_height,
-                                                    sdl, sys->display);
-                if (sys->overlay && !sys->overlay->hw_overlay && pass == 0) {
-                    /* Ignore non hardware overlay surface in first pass */
-                    SDL_FreeYUVOverlay(sys->overlay);
-                    sys->overlay = NULL;
-                }
-                if (sys->overlay) {
-                    /* We keep the surface locked forever */
-                    SDL_LockYUVOverlay(sys->overlay);
-
-                    fmt.i_chroma = vlc;
-                    sys->is_uv_swapped = vlc_fourcc_AreUVPlanesSwapped(fmt.i_chroma,
-                                                                       vd->fmt.i_chroma);
-                    if (sys->is_uv_swapped)
-                        fmt.i_chroma = vd->fmt.i_chroma;
-                    break;
-                }
-            }
-        }
-    } else {
-        msg_Warn(vd, "SDL overlay disabled by the user");
-    }
-
-    /* */
-    vout_display_cfg_t place_cfg = *vd->cfg;
-    place_cfg.display.width  = display_width;
-    place_cfg.display.height = display_height;
-    vout_display_PlacePicture(&sys->place, &vd->source, &place_cfg, !sys->overlay);
-
-    /* If no overlay, fallback to software output */
-    if (!sys->overlay) {
-        /* */
-        switch (sys->display->format->BitsPerPixel) {
-        case 8:
-            fmt.i_chroma = VLC_CODEC_RGB8;
-            break;
-        case 15:
-            fmt.i_chroma = VLC_CODEC_RGB15;
-            break;
-        case 16:
-            fmt.i_chroma = VLC_CODEC_RGB16;
-            break;
-        case 24:
-            fmt.i_chroma = VLC_CODEC_RGB24;
-            break;
-        case 32:
-            fmt.i_chroma = VLC_CODEC_RGB32;
-            break;
-        default:
-            msg_Err(vd, "unknown screen depth %i",
-                    sys->display->format->BitsPerPixel);
-            goto error;
-        }
-
-        /* All we have is an RGB image with square pixels */
-        fmt.i_width  = display_width;
-        fmt.i_height = display_height;
-        fmt.i_rmask = sys->display->format->Rmask;
-        fmt.i_gmask = sys->display->format->Gmask;
-        fmt.i_bmask = sys->display->format->Bmask;
-
-        info.has_pictures_invalid = true;
-    }
-
-    if (vd->cfg->display.title)
-        SDL_WM_SetCaption(vd->cfg->display.title,
-                          vd->cfg->display.title);
-    else if (!sys->overlay)
-        SDL_WM_SetCaption(VOUT_TITLE " (software RGB SDL output)",
-                          VOUT_TITLE " (software RGB SDL output)");
-    else if (sys->overlay->hw_overlay)
-        SDL_WM_SetCaption(VOUT_TITLE " (hardware YUV SDL output)",
-                          VOUT_TITLE " (hardware YUV SDL output)");
-    else
-        SDL_WM_SetCaption(VOUT_TITLE " (software YUV SDL output)",
-                          VOUT_TITLE " (software YUV SDL output)");
-
-    /* Setup events */
-    SDL_EventState(SDL_KEYUP, SDL_IGNORE);               /* ignore keys up */
-
-    /* Setup vout_display now that everything is fine */
-    vd->fmt = fmt;
-    vd->info = info;
-
-    vd->pool    = Pool;
-    vd->prepare = NULL;
-    vd->display = PictureDisplay;
-    vd->control = Control;
-    vd->manage  = Manage;
-
-    /* */
-    vout_display_SendEventDisplaySize(vd, display_width, display_height);
-    return VLC_SUCCESS;
-
-error:
-    msg_Err(vd, "cannot set up SDL (%s)", SDL_GetError());
-
-    if (sys->display) {
-        SDL_UnlockSurface(sys->display);
-        SDL_FreeSurface(sys->display);
-    }
-
-    vlc_mutex_lock(&sdl_lock);
-    SDL_QuitSubSystem(SDL_INIT_VIDEO);
-    vlc_mutex_unlock(&sdl_lock);
-
-    free(sys);
-    return VLC_EGENERIC;
-}
-
-/**
- * Close a SDL video output
- */
-static void Close(vlc_object_t *object)
-{
-    vout_display_t *vd = (vout_display_t *)object;
-    vout_display_sys_t *sys = vd->sys;
-
-    if (sys->pool)
-        picture_pool_Release(sys->pool);
-
-    if (sys->overlay) {
-        SDL_LockYUVOverlay(sys->overlay);
-        SDL_FreeYUVOverlay(sys->overlay);
-    }
-    SDL_UnlockSurface (sys->display);
-    SDL_FreeSurface(sys->display);
-
-    vlc_mutex_lock(&sdl_lock);
-    SDL_QuitSubSystem(SDL_INIT_VIDEO);
-    vlc_mutex_unlock(&sdl_lock);
-
-    free(sys);
-}
-
-/**
- * Return a pool of direct buffers
- */
-static picture_pool_t *Pool(vout_display_t *vd, unsigned count)
-{
-    vout_display_sys_t *sys = vd->sys;
-    VLC_UNUSED(count);
-
-    if (!sys->pool) {
-        picture_resource_t rsc;
-
-        memset(&rsc, 0, sizeof(rsc));
-
-        if (sys->overlay) {
-            SDL_Overlay *ol = sys->overlay;
-
-            for (int i = 0; i < ol->planes; i++) {
-                rsc.p[i].p_pixels = ol->pixels[ i > 0 && sys->is_uv_swapped ? (3-i) : i];
-                rsc.p[i].i_pitch  = ol->pitches[i > 0 && sys->is_uv_swapped ? (3-i) : i];
-                rsc.p[i].i_lines  = ol->h;
-                if (ol->format == SDL_YV12_OVERLAY ||
-                    ol->format == SDL_IYUV_OVERLAY)
-                    rsc.p[i].i_lines /= 2;
-
-            }
-        } else {
-            const int x = sys->place.x;
-            const int y = sys->place.y;
-
-            SDL_Surface *sf = sys->display;
-            SDL_FillRect(sf, NULL, 0);
-
-            assert(x >= 0 && y >= 0);
-            rsc.p[0].p_pixels = (uint8_t*)sf->pixels + y * sf->pitch + x * ((sf->format->BitsPerPixel + 7) / 8);
-            rsc.p[0].i_pitch  = sf->pitch;
-            rsc.p[0].i_lines  = vd->fmt.i_height;
-        }
-
-        picture_t *picture = picture_NewFromResource(&vd->fmt, &rsc);;
-        if (!picture)
-            return NULL;
-
-        sys->pool = picture_pool_New(1, &picture);
-    }
-
-    return sys->pool;
-}
-
-/**
- * Display a picture
- */
-static void PictureDisplay(vout_display_t *vd, picture_t *p_pic, subpicture_t *p_subpicture)
-{
-    vout_display_sys_t *sys = vd->sys;
-
-    if (sys->overlay) {
-        SDL_Rect disp;
-        disp.x = sys->place.x;
-        disp.y = sys->place.y;
-        disp.w = sys->place.width;
-        disp.h = sys->place.height;
-
-        SDL_UnlockYUVOverlay(sys->overlay);
-        SDL_DisplayYUVOverlay(sys->overlay , &disp);
-        SDL_LockYUVOverlay(sys->overlay);
-    } else {
-        SDL_Flip(sys->display);
-    }
-
-    picture_Release(p_pic);
-    VLC_UNUSED(p_subpicture);
-}
-
-
-/**
- * Control for vout display
- */
-static int Control(vout_display_t *vd, int query, va_list args)
-{
-    vout_display_sys_t *sys = vd->sys;
-
-    switch (query)
-    {
-    case VOUT_DISPLAY_HIDE_MOUSE:
-        SDL_ShowCursor(0);
-        return VLC_SUCCESS;
-
-    case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE: {
-        const vout_display_cfg_t *cfg = va_arg(args, const vout_display_cfg_t *);
-
-        /* */
-        sys->display = SDL_SetVideoMode(cfg->display.width,
-                                        cfg->display.height,
-                                        sys->display_bpp, sys->display_flags);
-        if (!sys->display) {
-            sys->display = SDL_SetVideoMode(vd->cfg->display.width,
-                                            vd->cfg->display.height,
-                                            sys->display_bpp, sys->display_flags);
-            return VLC_EGENERIC;
-        }
-        if (sys->overlay)
-            vout_display_PlacePicture(&sys->place, &vd->source, cfg, !sys->overlay);
-        else
-            vout_display_SendEventPicturesInvalid(vd);
-        return VLC_SUCCESS;
-    }
-    case VOUT_DISPLAY_CHANGE_FULLSCREEN: {
-        bool fs = va_arg(args, int);
-
-        /* Fix flags */
-        sys->display_flags &= ~(SDL_FULLSCREEN | SDL_RESIZABLE);
-        sys->display_flags |= fs ? SDL_FULLSCREEN : SDL_RESIZABLE;
-
-        if (sys->overlay)
-            sys->display = SDL_SetVideoMode(sys->desktop_width, sys->desktop_height,
-                                            sys->display_bpp, sys->display_flags);
-
-        vout_display_SendEventDisplaySize(vd, sys->desktop_width, sys->desktop_height);
-        return VLC_SUCCESS;
-    }
-    case VOUT_DISPLAY_CHANGE_ZOOM:
-    case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
-    case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT: {
-        const vout_display_cfg_t *cfg;
-
-        if (query == VOUT_DISPLAY_CHANGE_SOURCE_ASPECT) {
-            cfg = vd->cfg;
-        } else {
-            cfg = va_arg(args, const vout_display_cfg_t *);
-        }
-        if (sys->overlay) {
-            sys->display = SDL_SetVideoMode(cfg->display.width, cfg->display.height,
-                                            sys->display_bpp, sys->display_flags);
-
-            vout_display_PlacePicture(&sys->place, &vd->source, cfg, !sys->overlay);
-        } else {
-            vout_display_SendEventPicturesInvalid(vd);
-        }
-        return VLC_SUCCESS;
-    }
-
-    case VOUT_DISPLAY_RESET_PICTURES: {
-        /* */
-        assert(!sys->overlay);
-
-        /* */
-        if (sys->pool)
-            picture_pool_Release(sys->pool);
-        sys->pool = NULL;
-
-        vout_display_PlacePicture(&sys->place, &vd->source, vd->cfg, !sys->overlay);
-
-        /* */
-        vd->fmt.i_width  = sys->place.width;
-        vd->fmt.i_height = sys->place.height;
-        return VLC_SUCCESS;
-    }
-
-    case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
-        /* I don't think it is possible to support with SDL:
-         * - crop
-         */
-        return VLC_EGENERIC;
-
-    default:
-        msg_Err(vd, "Unsupported query in vout display SDL");
-        return VLC_EGENERIC;
-    }
-}
-
-/**
- * Proccess pending event
- */
-static void Manage(vout_display_t *vd)
-{
-    vout_display_sys_t *sys = vd->sys;
-    SDL_Event event;
-
-    /* */
-    while (SDL_PollEvent(&event)) {
-        switch (event.type) {
-        case SDL_QUIT:
-            vout_display_SendEventClose(vd);
-            break;
-
-        case SDL_KEYDOWN: {
-            /* convert the key if possible */
-            int key = ConvertKey(event.key.keysym.sym);
-
-            if (!key) {
-                /* Find the right caracter */
-                if ((event.key.keysym.unicode & 0xff80) == 0) {
-                    key = event.key.keysym.unicode & 0x7f;
-                    /* FIXME: find a better solution than this
-                              hack to find the right caracter */
-                    if (key >= 1 && key <= 26)
-                        key += 96;
-                    else if (key >= 65 && key <= 90)
-                        key += 32;
-                }
-            }
-            if (!key)
-                break;
-
-            if (event.key.keysym.mod & KMOD_SHIFT)
-                key |= KEY_MODIFIER_SHIFT;
-            if (event.key.keysym.mod & KMOD_CTRL)
-                key |= KEY_MODIFIER_CTRL;
-            if (event.key.keysym.mod & KMOD_ALT)
-                key |= KEY_MODIFIER_ALT;
-            vout_display_SendEventKey(vd, key);
-            break;
-        }
-
-        case SDL_MOUSEBUTTONDOWN:
-        case SDL_MOUSEBUTTONUP: {
-            static const struct { int sdl; int vlc; } buttons[] = {
-                { SDL_BUTTON_LEFT,      MOUSE_BUTTON_LEFT },
-                { SDL_BUTTON_MIDDLE,    MOUSE_BUTTON_CENTER },
-                { SDL_BUTTON_RIGHT,     MOUSE_BUTTON_RIGHT },
-                { SDL_BUTTON_WHEELUP,   MOUSE_BUTTON_WHEEL_UP },
-                { SDL_BUTTON_WHEELDOWN, MOUSE_BUTTON_WHEEL_DOWN },
-                { -1, -1 },
-            };
-
-            SDL_ShowCursor(1);
-            for (int i = 0; buttons[i].sdl != -1; i++) {
-                if (buttons[i].sdl == event.button.button) {
-                    if (event.type == SDL_MOUSEBUTTONDOWN)
-                        vout_display_SendEventMousePressed(vd, buttons[i].vlc);
-                    else
-                        vout_display_SendEventMouseReleased(vd, buttons[i].vlc);
-                }
-            }
-            break;
-        }
-
-        case SDL_MOUSEMOTION: {
-            if (sys->place.width <= 0 || sys->place.height <= 0)
-                break;
-
-            SDL_ShowCursor(1);
-
-            vout_display_SendMouseMovedDisplayCoordinates(vd, ORIENT_NORMAL,
-                                                          event.motion.x, event.motion.y,
-                                                          &sys->place);
-            break;
-        }
-
-        case SDL_VIDEORESIZE:
-            vout_display_SendEventDisplaySize(vd, event.resize.w, event.resize.h);
-            break;
-
-        default:
-            break;
-        }
-    }
-
-}
-
-static const struct {
-    SDLKey sdl_key;
-    int    vlckey;
-
-} sdlkeys_to_vlckeys[] = {
-    { SDLK_F1,  KEY_F1 },
-    { SDLK_F2,  KEY_F2 },
-    { SDLK_F3,  KEY_F3 },
-    { SDLK_F4,  KEY_F4 },
-    { SDLK_F5,  KEY_F5 },
-    { SDLK_F6,  KEY_F6 },
-    { SDLK_F7,  KEY_F7 },
-    { SDLK_F8,  KEY_F8 },
-    { SDLK_F9,  KEY_F9 },
-    { SDLK_F10, KEY_F10 },
-    { SDLK_F11, KEY_F11 },
-    { SDLK_F12, KEY_F12 },
-
-    { SDLK_RETURN, KEY_ENTER },
-    { SDLK_KP_ENTER, KEY_ENTER },
-    { SDLK_SPACE, ' ' },
-    { SDLK_ESCAPE, KEY_ESC },
-
-    { SDLK_MENU, KEY_MENU },
-    { SDLK_LEFT, KEY_LEFT },
-    { SDLK_RIGHT, KEY_RIGHT },
-    { SDLK_UP, KEY_UP },
-    { SDLK_DOWN, KEY_DOWN },
-
-    { SDLK_HOME, KEY_HOME },
-    { SDLK_END, KEY_END },
-    { SDLK_PAGEUP, KEY_PAGEUP },
-    { SDLK_PAGEDOWN,  KEY_PAGEDOWN },
-
-    { SDLK_INSERT, KEY_INSERT },
-    { SDLK_DELETE, KEY_DELETE },
-    /*TODO: find a equivalent for SDL 
-    { , KEY_MEDIA_NEXT_TRACK }
-    { , KEY_MEDIA_PREV_TRACK }
-    { , KEY_VOLUME_MUTE }
-    { , KEY_VOLUME_DOWN }
-    { , KEY_VOLUME_UP }
-    { , KEY_MEDIA_PLAY_PAUSE }
-    { , KEY_MEDIA_PLAY_PAUSE }*/
-
-    { 0, 0 }
-};
-
-static int ConvertKey(SDLKey sdl_key)
-{
-    for (int i = 0; sdlkeys_to_vlckeys[i].sdl_key != 0; i++) {
-        if (sdlkeys_to_vlckeys[i].sdl_key == sdl_key)
-            return sdlkeys_to_vlckeys[i].vlckey;
-    }
-    return 0;
-}
-
diff --git a/po/POTFILES.in b/po/POTFILES.in
index 0aa9f039ee..49b0679632 100644
--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -1140,7 +1140,6 @@ modules/video_output/win32/directdraw.c
 modules/video_output/win32/events.c
 modules/video_output/win32/glwin32.c
 modules/video_output/win32/wingdi.c
-modules/video_output/sdl.c
 modules/video_output/vdummy.c
 modules/video_output/vmem.c
 modules/video_output/wayland/shell.c
-- 
2.13.3



More information about the vlc-devel mailing list