From 4a4dc6dfc23efd39f9418b8bd551671a27a8429a Mon Sep 17 00:00:00 2001 From: =?utf8?q?R=C3=A9mi=20Denis-Courmont?= Date: Sat, 23 May 2009 20:18:29 +0300 Subject: [PATCH] Merge intf_Create() and intf_RunThread() Interfaces do not have an event handler that would justify the separation. Indeed, no caller did anything in-between. Also, the "primary" reference to the interface object belongs to the main thread because of the libvlc cleanup procedure. Therefore, the interface thread pointer really should not be returned to the creator. Note that this does not really fix the small race condition but it conceals it within intf_Create() and libvlc_InternalCleanup(). --- include/vlc_interface.h | 5 ++-- modules/access/vcdx/access.c | 4 --- modules/control/ntservice.c | 14 +-------- modules/control/rc.c | 13 +------- src/interface/interface.c | 58 ++++++++++-------------------------- src/libvlc.c | 16 ++-------- src/libvlccore.sym | 3 +- 7 files changed, 23 insertions(+), 90 deletions(-) diff --git a/include/vlc_interface.h b/include/vlc_interface.h index b59bd9da1e..e368d0bad9 100644 --- a/include/vlc_interface.h +++ b/include/vlc_interface.h @@ -98,9 +98,8 @@ struct intf_dialog_args_t /***************************************************************************** * Prototypes *****************************************************************************/ -#define intf_Create(a,b) __intf_Create(VLC_OBJECT(a),b) -VLC_EXPORT( intf_thread_t *, __intf_Create, ( vlc_object_t *, const char * ) ); -VLC_EXPORT( int, intf_RunThread, ( intf_thread_t * ) ); +VLC_EXPORT( int, intf_Create, ( vlc_object_t *, const char * ) ); +#define intf_Create(a,b) intf_Create(VLC_OBJECT(a),b) VLC_EXPORT( void, intf_StopThread, ( intf_thread_t * ) ); #define intf_Eject(a,b) __intf_Eject(VLC_OBJECT(a),b) diff --git a/modules/access/vcdx/access.c b/modules/access/vcdx/access.c index fa3c427093..e0eb969905 100644 --- a/modules/access/vcdx/access.c +++ b/modules/access/vcdx/access.c @@ -967,10 +967,6 @@ VCDOpen ( vlc_object_t *p_this ) #endif p_vcdplayer->p_access = p_access; -#ifdef FIXED - intf_RunThread( p_vcdplayer->p_intf ); -#endif - free( psz_source ); return VLC_SUCCESS; diff --git a/modules/control/ntservice.c b/modules/control/ntservice.c index 62b097b233..e7104cba01 100644 --- a/modules/control/ntservice.c +++ b/modules/control/ntservice.c @@ -309,26 +309,14 @@ static void WINAPI ServiceDispatch( DWORD numArgs, char **args ) if( asprintf( &psz_temp, "%s,none", psz_module ) != -1 ) { - intf_thread_t *p_new_intf; - /* Try to create the interface */ - p_new_intf = intf_Create( p_intf, psz_temp ); - if( p_new_intf == NULL ) + if( intf_Create( p_intf, psz_temp ) ) { msg_Err( p_intf, "interface \"%s\" initialization failed", psz_temp ); free( psz_temp ); continue; } - - /* Try to run the interface */ - if( intf_RunThread( p_new_intf ) ) - { - vlc_object_detach( p_new_intf ); - vlc_object_release( p_new_intf ); - msg_Err( p_intf, "interface \"%s\" cannot run", psz_temp ); - } - free( psz_temp ); } } diff --git a/modules/control/rc.c b/modules/control/rc.c index c1602eab17..843096d043 100644 --- a/modules/control/rc.c +++ b/modules/control/rc.c @@ -1525,19 +1525,8 @@ static int Intf( vlc_object_t *p_this, char const *psz_cmd, vlc_value_t oldval, vlc_value_t newval, void *p_data ) { VLC_UNUSED(psz_cmd); VLC_UNUSED(oldval); VLC_UNUSED(p_data); - intf_thread_t *p_newintf = NULL; - p_newintf = intf_Create( p_this->p_libvlc, newval.psz_string ); - if( p_newintf ) - { - if( intf_RunThread( p_newintf ) ) - { - vlc_object_detach( p_newintf ); - vlc_object_release( p_newintf ); - } - } - - return VLC_SUCCESS; + return intf_Create( p_this->p_libvlc, newval.psz_string ); } static int Volume( vlc_object_t *p_this, char const *psz_cmd, diff --git a/src/interface/interface.c b/src/interface/interface.c index c48986a1f6..105783ccb0 100644 --- a/src/interface/interface.c +++ b/src/interface/interface.c @@ -72,22 +72,26 @@ static void intf_Destroy( vlc_object_t *obj ) } +#undef intf_Create /** - * Create the interface, and prepare it for main loop. It opens ouput device - * and creates specific interfaces. Sends its own error messages. + * Create and start an interface. * * @param p_this the calling vlc_object_t * @param psz_module a preferred interface module - * @return a pointer to the created interface thread, NULL on error + * @return VLC_SUCCESS or an error code */ -intf_thread_t* __intf_Create( vlc_object_t *p_this, const char *psz_module ) +int intf_Create( vlc_object_t *p_this, const char *psz_module ) { intf_thread_t * p_intf; /* Allocate structure */ p_intf = vlc_object_create( p_this, VLC_OBJECT_INTF ); if( !p_intf ) - return NULL; + return VLC_ENOMEM; + + /* Attach interface to its parent object */ + vlc_object_attach( p_intf, p_this ); + vlc_object_set_destructor( p_intf, intf_Destroy ); #if defined( __APPLE__ ) || defined( WIN32 ) p_intf->b_should_run_on_first_thread = false; #endif @@ -102,31 +106,13 @@ intf_thread_t* __intf_Create( vlc_object_t *p_this, const char *psz_module ) free( psz_tmp ); free( psz_parser ); p_intf->p_module = module_need( p_intf, "interface", p_intf->psz_intf, true ); - if( p_intf->p_module == NULL ) { msg_Err( p_intf, "no suitable interface module" ); - free( p_intf->psz_intf ); vlc_object_release( p_intf ); - return NULL; + return VLC_EGENERIC; } - /* Attach interface to its parent object */ - vlc_object_attach( p_intf, p_this ); - vlc_object_set_destructor( p_intf, intf_Destroy ); - - return p_intf; -} - - -/** - * Starts and runs the interface thread. - * - * @param p_intf the interface thread - * @return VLC_SUCCESS on success, an error number else - */ -int intf_RunThread( intf_thread_t *p_intf ) -{ #if defined( __APPLE__ ) || defined( WIN32 ) /* Hack to get Mac OS X Cocoa runtime running * (it needs access to the main thread) */ @@ -136,7 +122,8 @@ int intf_RunThread( intf_thread_t *p_intf ) VLC_THREAD_PRIORITY_LOW ) ) { msg_Err( p_intf, "cannot spawn libvlc death monitoring thread" ); - return VLC_EGENERIC; + vlc_object_release( p_intf ); + return VLC_ENOMEM; } RunInterface( VLC_OBJECT(p_intf) ); @@ -156,6 +143,7 @@ int intf_RunThread( intf_thread_t *p_intf ) VLC_THREAD_PRIORITY_LOW ) ) { msg_Err( p_intf, "cannot spawn interface thread" ); + vlc_object_release( p_intf ); return VLC_EGENERIC; } @@ -250,30 +238,16 @@ static int AddIntfCallback( vlc_object_t *p_this, char const *psz_cmd, vlc_value_t oldval, vlc_value_t newval, void *p_data ) { (void)psz_cmd; (void)oldval; (void)p_data; - intf_thread_t *p_intf; char* psz_intf; /* Try to create the interface */ if( asprintf( &psz_intf, "%s,none", newval.psz_string ) == -1 ) return VLC_ENOMEM; - p_intf = intf_Create( p_this->p_libvlc, psz_intf ); + int ret = intf_Create( p_this->p_libvlc, psz_intf ); free( psz_intf ); - if( p_intf == NULL ) - { + if( ret ) msg_Err( p_this, "interface \"%s\" initialization failed", newval.psz_string ); - return VLC_EGENERIC; - } - - /* Try to run the interface */ - if( intf_RunThread( p_intf ) != VLC_SUCCESS ) - { - vlc_object_detach( p_intf ); - vlc_object_release( p_intf ); - return VLC_EGENERIC; - } - - return VLC_SUCCESS; + return ret; } - diff --git a/src/libvlc.c b/src/libvlc.c index 0fb54b0246..e9e0b0435e 100644 --- a/src/libvlc.c +++ b/src/libvlc.c @@ -1138,7 +1138,6 @@ void libvlc_InternalDestroy( libvlc_int_t *p_libvlc ) */ int libvlc_InternalAddIntf( libvlc_int_t *p_libvlc, char const *psz_module ) { - int i_err; intf_thread_t *p_intf = NULL; if( !p_libvlc ) @@ -1164,25 +1163,14 @@ int libvlc_InternalAddIntf( libvlc_int_t *p_libvlc, char const *psz_module ) } /* Try to create the interface */ - p_intf = intf_Create( p_libvlc, psz_module ? psz_module : "$intf" ); - if( p_intf == NULL ) + if( intf_Create( p_libvlc, psz_module ? psz_module : "$intf" ) ) { msg_Err( p_libvlc, "interface \"%s\" initialization failed", psz_module ); return VLC_EGENERIC; } - - /* Try to run the interface */ - i_err = intf_RunThread( p_intf ); - if( i_err ) - { - vlc_object_detach( p_intf ); - vlc_object_release( p_intf ); - return i_err; - } - return VLC_SUCCESS; -}; +} static vlc_mutex_t exit_lock = VLC_STATIC_MUTEX; diff --git a/src/libvlccore.sym b/src/libvlccore.sym index 3f1df055f8..0174274404 100644 --- a/src/libvlccore.sym +++ b/src/libvlccore.sym @@ -203,9 +203,8 @@ input_SplitMRL input_Start input_Stop input_vaControl -__intf_Create +intf_Create __intf_Eject -intf_RunThread intf_StopThread IsUTF8 libvlc_InternalAddIntf -- 2.39.2