[vlc-devel] [PATCH v3 17/18] video_output: remove unused control API

Steve Lhomme robux4 at ycbcr.xyz
Tue Aug 18 16:58:44 CEST 2020


---
 src/Makefile.am                 |  2 -
 src/video_output/control.c      | 86 ---------------------------------
 src/video_output/control.h      | 64 ------------------------
 src/video_output/video_output.c |  4 +-
 4 files changed, 1 insertion(+), 155 deletions(-)
 delete mode 100644 src/video_output/control.c
 delete mode 100644 src/video_output/control.h

diff --git a/src/Makefile.am b/src/Makefile.am
index 9e7c2931d27..3bef0703e3e 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -306,8 +306,6 @@ libvlccore_la_SOURCES = \
 	audio_output/output.c \
 	audio_output/volume.c \
 	video_output/chrono.h \
-	video_output/control.c \
-	video_output/control.h \
 	video_output/display.c \
 	video_output/display.h \
 	video_output/inhibit.c \
diff --git a/src/video_output/control.c b/src/video_output/control.c
deleted file mode 100644
index 83a08043e3a..00000000000
--- a/src/video_output/control.c
+++ /dev/null
@@ -1,86 +0,0 @@
-/*****************************************************************************
- * control.c : vout internal control
- *****************************************************************************
- * Copyright (C) 2009 Laurent Aimar
- *
- * Authors: 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.
- *****************************************************************************/
-
-#ifdef HAVE_CONFIG_H
-# include "config.h"
-#endif
-
-#include <vlc_common.h>
-#include <vlc_vout.h>
-#include "vout_internal.h"
-#include "control.h"
-
-/* */
-void vout_control_cmd_Init(vout_control_cmd_t *cmd, int type)
-{
-    memset(cmd, 0, sizeof(*cmd));
-    cmd->type = type;
-}
-
-/* */
-void vout_control_Init(vout_control_t *ctrl)
-{
-    ctrl->can_sleep = true;
-    ARRAY_INIT(ctrl->cmd);
-}
-
-void vout_control_Clean(vout_control_t *ctrl)
-{
-    /* */
-    ARRAY_RESET(ctrl->cmd);
-}
-
-void vout_control_Push(vout_control_t *ctrl, vout_control_cmd_t *cmd)
-{
-    {
-        ARRAY_APPEND(ctrl->cmd, *cmd);
-    }
-}
-
-void vout_control_Wake(vout_control_t *ctrl)
-{
-    ctrl->can_sleep = false;
-}
-
-int vout_control_Pop(vout_control_t *ctrl, vout_control_cmd_t *cmd,
-                     vlc_tick_t deadline, vlc_cond_t *wait_request, vlc_mutex_t *lock)
-{
-    if (ctrl->cmd.i_size <= 0) {
-        /* Spurious wakeups are perfectly fine */
-        if (deadline != INVALID_DEADLINE && ctrl->can_sleep) {
-            vlc_cond_timedwait(wait_request, lock, deadline);
-        }
-    }
-
-    bool has_cmd;
-    if (ctrl->cmd.i_size > 0) {
-        has_cmd = true;
-        *cmd = ARRAY_VAL(ctrl->cmd, 0);
-        ARRAY_REMOVE(ctrl->cmd, 0);
-    } else {
-        has_cmd = false;
-        ctrl->can_sleep = true;
-    }
-
-    return has_cmd ? VLC_SUCCESS : VLC_EGENERIC;
-}
-
diff --git a/src/video_output/control.h b/src/video_output/control.h
deleted file mode 100644
index a5bbef577b6..00000000000
--- a/src/video_output/control.h
+++ /dev/null
@@ -1,64 +0,0 @@
-/*****************************************************************************
- * control.h : vout internal control
- *****************************************************************************
- * Copyright (C) 2009-2010 Laurent Aimar
- *
- * Authors: 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.
- *****************************************************************************/
-
-#ifndef LIBVLC_VOUT_INTERNAL_CONTROL_H
-#define LIBVLC_VOUT_INTERNAL_CONTROL_H
-
-#include <vlc_viewpoint.h>
-
-/* */
-enum {
-    VOUT_CONTROL_MOUSE_STATE,           /* vlc_mouse_t */
-};
-
-typedef struct {
-    int type;
-
-    union {
-        vlc_mouse_t mouse;
-    };
-} vout_control_cmd_t;
-
-void vout_control_cmd_Init(vout_control_cmd_t *, int type);
-
-typedef struct {
-    /* */
-    bool can_sleep;
-    DECL_ARRAY(vout_control_cmd_t) cmd;
-} vout_control_t;
-
-/* */
-void vout_control_Init(vout_control_t *);
-void vout_control_Clean(vout_control_t *);
-
-/* controls outside of the vout thread */
-void vout_control_WaitEmpty(vout_control_t *);
-
-void vout_control_Push(vout_control_t *, vout_control_cmd_t *);
-void vout_control_Wake(vout_control_t *);
-
-/* control inside of the vout thread */
-#define INVALID_DEADLINE   ((vlc_tick_t) INT64_MAX)
-
-int vout_control_Pop(vout_control_t *, vout_control_cmd_t *, vlc_tick_t deadline, vlc_cond_t *, vlc_mutex_t *);
-
-#endif
diff --git a/src/video_output/video_output.c b/src/video_output/video_output.c
index 8b3354bc5a7..c87a6031808 100644
--- a/src/video_output/video_output.c
+++ b/src/video_output/video_output.c
@@ -60,7 +60,6 @@
 #include "../clock/clock.h"
 #include "statistic.h"
 #include "chrono.h"
-#include "control.h"
 
 typedef struct vout_thread_sys_t
 {
@@ -124,7 +123,6 @@ typedef struct vout_thread_sys_t
     vlc_cond_t      has_pictures;
     bool            terminate;
     DECL_ARRAY(vlc_mouse_t) mouse_events;
-    vout_control_t  control;
     vlc_thread_t    thread;
 
     struct {
@@ -196,6 +194,7 @@ typedef struct vout_thread_sys_t
 #define VOUT_THREAD_TO_SYS(vout) \
     container_of(vout, vout_thread_sys_t, obj.obj)
 
+#define INVALID_DEADLINE   ((vlc_tick_t) INT64_MAX)
 
 /* Maximum delay between 2 displayed pictures.
  * XXX it is needed for now but should be removed in the long term.
@@ -2239,7 +2238,6 @@ int vout_Request(const vout_configuration_t *cfg, vlc_video_context *vctx, input
 
     sys->original = original;
 
-    sys->delay = 0;
     sys->rate = 1.f;
     sys->clock = cfg->clock;
     sys->delay = 0;
-- 
2.26.2



More information about the vlc-devel mailing list