[vlc-devel] [PATCH 21/21] libvlc: merge libvlc_video_direct3d_set_callbacks into libvlc_video_set_output_callbacks
Steve Lhomme
robux4 at ycbcr.xyz
Wed Feb 5 16:13:29 CET 2020
The GetProcAddress callback is always NULL in that case.
---
doc/libvlc/d3d11_player.cpp | 8 ++--
doc/libvlc/d3d9_player.c | 8 ++--
include/vlc/libvlc_media_player.h | 63 ++++---------------------------
lib/libvlc.sym | 1 -
lib/media_player.c | 34 +----------------
5 files changed, 18 insertions(+), 96 deletions(-)
diff --git a/doc/libvlc/d3d11_player.cpp b/doc/libvlc/d3d11_player.cpp
index c9bf86a3098..19244408dd7 100644
--- a/doc/libvlc/d3d11_player.cpp
+++ b/doc/libvlc/d3d11_player.cpp
@@ -581,10 +581,10 @@ int WINAPI WinMain(HINSTANCE hInstance,
// DON'T use with callbacks libvlc_media_player_set_hwnd(p_mp, hWnd);
/* Tell VLC to render into our D3D11 environment */
- libvlc_video_direct3d_set_callbacks( p_mp, libvlc_video_direct3d_engine_d3d11,
- Setup_cb, Cleanup_cb, Resize_cb, UpdateOutput_cb, Swap_cb, StartRendering_cb,
- nullptr, SelectPlane_cb,
- &Context );
+ libvlc_video_set_output_callbacks( p_mp, libvlc_video_direct3d_engine_d3d11,
+ Setup_cb, Cleanup_cb, Resize_cb, UpdateOutput_cb, Swap_cb, StartRendering_cb,
+ nullptr, nullptr, SelectPlane_cb,
+ &Context );
libvlc_media_player_play( p_mp );
diff --git a/doc/libvlc/d3d9_player.c b/doc/libvlc/d3d9_player.c
index 3e15caaedc8..6ce33181301 100644
--- a/doc/libvlc/d3d9_player.c
+++ b/doc/libvlc/d3d9_player.c
@@ -376,10 +376,10 @@ int WINAPI WinMain(HINSTANCE hInstance,
// DON'T use with callbacks libvlc_media_player_set_hwnd(p_mp, hWnd);
/* Tell VLC to render into our D3D9 environment */
- libvlc_video_direct3d_set_callbacks( p_mp, libvlc_video_direct3d_engine_d3d9,
- Setup_cb, Cleanup_cb, Resize_cb, UpdateOutput_cb, Swap_cb, StartRendering_cb,
- NULL, NULL,
- &Context );
+ libvlc_video_set_output_callbacks( p_mp, libvlc_video_direct3d_engine_d3d9,
+ Setup_cb, Cleanup_cb, Resize_cb, UpdateOutput_cb, Swap_cb, StartRendering_cb,
+ NULL, NULL, NULL,
+ &Context );
libvlc_media_player_play( p_mp );
diff --git a/include/vlc/libvlc_media_player.h b/include/vlc/libvlc_media_player.h
index 9dc0056abf6..cd02fe01e71 100644
--- a/include/vlc/libvlc_media_player.h
+++ b/include/vlc/libvlc_media_player.h
@@ -521,10 +521,9 @@ typedef struct
* Setup the rendering environment.
*
* \param opaque private pointer passed to the @a libvlc_video_set_output_callbacks()
- * or @a libvlc_video_direct3d_set_callbacks()
* on input. The callback can change this value on output to be
- * passed to all the other callbacks set on @a libvlc_video_set_output_callbacks(). [IN/OUT]
- * or @a libvlc_video_set_output_callbacks(). [IN/OUT]
+ * passed to all the other callbacks set on @a libvlc_video_set_output_callbacks().
+ * [IN/OUT]
* \param cfg requested configuration of the video device [IN]
* \param out libvlc_video_setup_device_info_t* to fill [OUT]
* \return true on success
@@ -674,6 +673,10 @@ typedef void (*libvlc_video_frameMetadata_cb)(void* opaque, libvlc_video_metadat
typedef enum libvlc_video_engine_t {
libvlc_video_engine_opengl,
libvlc_video_engine_gles2,
+ /** Direct3D11 rendering engine */
+ libvlc_video_direct3d_engine_d3d11,
+ /** Direct3D9 rendering engine */
+ libvlc_video_direct3d_engine_d3d9,
} libvlc_video_engine_t;
/** Set the callback to call when the host app resizes the rendering area.
@@ -703,7 +706,7 @@ typedef void( *libvlc_video_output_set_resize_cb )( void *opaque,
* \note This is only used with \ref libvlc_video_direct3d_engine_d3d11.
*
* The host should call OMSetRenderTargets for Direct3D11. If this callback is
- * not used (set to NULL in @a libvlc_video_direct3d_set_callbacks()) OMSetRenderTargets
+ * not used (set to NULL in @a libvlc_video_set_output_callbacks()) OMSetRenderTargets
* has to be set during the @a libvlc_video_makeCurrent_cb()
* entering call.
*
@@ -719,8 +722,6 @@ typedef bool( *libvlc_video_output_select_plane_cb )( void *opaque, size_t plane
* \warning VLC will perform video rendering in its own thread and at its own rate,
* You need to provide your own synchronisation mechanism.
*
- * OpenGL context need to be created before playing a media.
- *
* \param mp the media player
* \param engine the GPU engine to use
* \param setup_cb callback called to initialize user data
@@ -728,7 +729,7 @@ typedef bool( *libvlc_video_output_select_plane_cb )( void *opaque, size_t plane
* \param resize_cb callback to set the resize callback
* \param update_output_cb callback to get the rendering format of the host (cannot be NULL)
* \param swap_cb callback called after rendering a video frame (cannot be NULL)
- * \param makeCurrent_cb callback called to enter/leave the opengl context (cannot be NULL for \ref libvlc_video_engine_opengl and for \ref libvlc_video_engine_gles2)
+ * \param makeCurrent_cb callback called to enter/leave the opengl context (cannot be NULL)
* \param getProcAddress_cb opengl function loading callback (cannot be NULL for \ref libvlc_video_engine_opengl and for \ref libvlc_video_engine_gles2)
* \param metadata_cb callback to provide frame metadata (D3D11 only)
* \param select_plane_cb callback to select different D3D11 rendering targets
@@ -752,54 +753,6 @@ bool libvlc_video_set_output_callbacks( libvlc_media_player_t *mp,
libvlc_video_output_select_plane_cb select_plane_cb,
void* opaque );
-
-/**
- * Enumeration of the Video engine to be used on output.
- * can be passed to @a libvlc_video_direct3d_set_callbacks
- */
-typedef enum libvlc_video_direct3d_engine_t {
- /** Direct3D11 rendering engine */
- libvlc_video_direct3d_engine_d3d11,
- /** Direct3D9 rendering engine */
- libvlc_video_direct3d_engine_d3d9,
-} libvlc_video_direct3d_engine_t;
-
-/**
- * Set callbacks and data to render decoded video to a custom Direct3D output
- *
- * \warning VLC will perform video rendering in its own thread and at its own rate,
- * You need to provide your own synchronisation mechanism.
- *
- * \param mp the media player
- * \param engine the GPU engine to use
- * \param setup_cb callback to setup and return the device to use (cannot be NULL)
- * \param cleanup_cb callback to cleanup the device given by the \ref setup_cb callback
- * \param resize_cb callback to set the resize callback
- * \param update_output_cb callback to notify of the source format and get the
- * rendering format used by the host (cannot be NULL)
- * \param swap_cb callback to tell the host it should display the rendered picture (cannot be NULL)
- * \param makeCurrent_cb callback to tell the host the rendering is starting/ended (cannot be NULL)
- * \param metadata_cb callback to provide frame metadata (D3D11 only)
- * \param select_plane_cb callback to select different D3D11 rendering targets
- * \param opaque private pointer passed to the \ref cleanup_cb callback
- *
- * \retval true Direct3D selected and callbacks set
- * \retval false Direct3D version, callbacks not set
- * \version LibVLC 4.0.0 or later
- */
-LIBVLC_API
-bool libvlc_video_direct3d_set_callbacks( libvlc_media_player_t *mp,
- libvlc_video_direct3d_engine_t engine,
- libvlc_video_output_setup_cb setup_cb,
- libvlc_video_output_cleanup_cb cleanup_cb,
- libvlc_video_output_set_resize_cb resize_cb,
- libvlc_video_update_output_cb update_output_cb,
- libvlc_video_swap_cb swap_cb,
- libvlc_video_makeCurrent_cb makeCurrent_cb,
- libvlc_video_frameMetadata_cb metadata_cb,
- libvlc_video_output_select_plane_cb select_plane_cb,
- void* opaque );
-
/**
* Set the NSView handler where the media player should render its video output.
*
diff --git a/lib/libvlc.sym b/lib/libvlc.sym
index d8fb0806bca..67e4dcf7482 100644
--- a/lib/libvlc.sym
+++ b/lib/libvlc.sym
@@ -239,7 +239,6 @@ libvlc_video_set_deinterlace
libvlc_video_set_format
libvlc_video_set_format_callbacks
libvlc_video_set_output_callbacks
-libvlc_video_direct3d_set_callbacks
libvlc_video_set_key_input
libvlc_video_set_logo_int
libvlc_video_set_logo_string
diff --git a/lib/media_player.c b/lib/media_player.c
index 1ee29b96023..4da37c13c8a 100644
--- a/lib/media_player.c
+++ b/lib/media_player.c
@@ -1062,38 +1062,7 @@ bool libvlc_video_set_output_callbacks(libvlc_media_player_t *mp,
var_SetString ( mp, "vout", "gl" );
var_SetString ( mp, "gl", "vgl");
}
- else
- return false;
-
- var_SetAddress( mp, "vout-cb-opaque", opaque );
- var_SetAddress( mp, "vout-cb-setup", setup_cb );
- var_SetAddress( mp, "vout-cb-cleanup", cleanup_cb );
- var_SetAddress( mp, "vout-cb-resize-cb", resize_cb );
- var_SetAddress( mp, "vout-cb-update-output", update_output_cb );
- var_SetAddress( mp, "vout-cb-swap", swap_cb );
- var_SetAddress( mp, "vout-cb-get-proc-address", getProcAddress_cb );
- var_SetAddress( mp, "vout-cb-make-current", makeCurrent_cb );
- var_SetAddress( mp, "vout-cb-metadata", metadata_cb );
- var_SetAddress( mp, "vout-cb-select-plane", select_plane_cb );
- return true;
-}
-
-
-bool libvlc_video_direct3d_set_callbacks(libvlc_media_player_t *mp,
- libvlc_video_direct3d_engine_t engine,
- libvlc_video_output_setup_cb setup_cb,
- libvlc_video_output_cleanup_cb cleanup_cb,
- libvlc_video_output_set_resize_cb resize_cb,
- libvlc_video_update_output_cb update_output_cb,
- libvlc_video_swap_cb swap_cb,
- libvlc_video_makeCurrent_cb makeCurrent_cb,
- libvlc_video_frameMetadata_cb metadata_cb,
- libvlc_video_output_select_plane_cb select_plane_cb,
- void *opaque)
-{
- var_SetString( mp, "window", "wextern");
-
- if ( engine == libvlc_video_direct3d_engine_d3d11 )
+ else if ( engine == libvlc_video_direct3d_engine_d3d11 )
{
var_SetString ( mp, "vout", "direct3d11" );
var_SetString ( mp, "dec-dev", "d3d11-device" );
@@ -1112,6 +1081,7 @@ bool libvlc_video_direct3d_set_callbacks(libvlc_media_player_t *mp,
var_SetAddress( mp, "vout-cb-resize-cb", resize_cb );
var_SetAddress( mp, "vout-cb-update-output", update_output_cb );
var_SetAddress( mp, "vout-cb-swap", swap_cb );
+ var_SetAddress( mp, "vout-cb-get-proc-address", getProcAddress_cb );
var_SetAddress( mp, "vout-cb-make-current", makeCurrent_cb );
var_SetAddress( mp, "vout-cb-metadata", metadata_cb );
var_SetAddress( mp, "vout-cb-select-plane", select_plane_cb );
--
2.17.1
More information about the vlc-devel
mailing list