From fddff181aa82b3a8c9ba1fd7ca81da54b1f8fb45 Mon Sep 17 00:00:00 2001 From: =?utf8?q?R=C3=A9mi=20Denis-Courmont?= Date: Sat, 20 Sep 2008 16:48:24 +0300 Subject: [PATCH] Finally remove mis-designed vlc_object_get --- include/vlc_objects.h | 1 - src/libvlccore.sym | 1 - src/misc/objects.c | 42 +----------------------------------------- 3 files changed, 1 insertion(+), 43 deletions(-) diff --git a/include/vlc_objects.h b/include/vlc_objects.h index c408f56df8..fc3dc8b95b 100644 --- a/include/vlc_objects.h +++ b/include/vlc_objects.h @@ -82,7 +82,6 @@ VLC_EXPORT( void, __vlc_object_detach, ( vlc_object_t * ) ); #if defined (__GNUC__) && !defined __cplusplus __attribute__((deprecated)) #endif -VLC_EXPORT( void *, vlc_object_get, ( libvlc_int_t *, int ) ); VLC_EXPORT( void *, __vlc_object_find, ( vlc_object_t *, int, int ) ); VLC_EXPORT( vlc_object_t *, vlc_object_find_name, ( vlc_object_t *, const char *, int ) ); VLC_EXPORT( void, __vlc_object_yield, ( vlc_object_t * ) ); diff --git a/src/libvlccore.sym b/src/libvlccore.sym index f428bd1a7a..59c9c17484 100644 --- a/src/libvlccore.sym +++ b/src/libvlccore.sym @@ -475,7 +475,6 @@ __vlc_object_create __vlc_object_detach __vlc_object_find vlc_object_find_name -vlc_object_get __vlc_object_kill __vlc_object_lock __vlc_object_release diff --git a/src/misc/objects.c b/src/misc/objects.c index beced54437..f806c619d0 100644 --- a/src/misc/objects.c +++ b/src/misc/objects.c @@ -514,47 +514,7 @@ void __vlc_object_kill( vlc_object_t *p_this ) } -/** - * Find an object given its ID. - * - * This function looks for the object whose i_object_id field is i_id. - * This function is slow, and often used to hide bugs. Do not use it. - * If you need to retain reference to an object, yield the object pointer with - * vlc_object_yield(), use the pointer as your reference, and call - * vlc_object_release() when you're done. - */ -void * vlc_object_get( libvlc_int_t *p_anchor, int i_id ) -{ - vlc_object_t *obj = NULL; -#ifndef NDEBUG - int canc = vlc_savecancel (); - fprintf (stderr, "Use of deprecated vlc_object_get(%d) ", i_id); - vlc_backtrace (); - vlc_restorecancel (canc); -#endif - vlc_mutex_lock( &structure_lock ); - - for( obj = vlc_internals (p_anchor)->next; - obj != VLC_OBJECT (p_anchor); - obj = vlc_internals (obj)->next ) - { - if( obj->i_object_id == i_id ) - { - vlc_object_yield( obj ); - goto out; - } - } - obj = NULL; -#ifndef NDEBUG - fprintf (stderr, "Object %d does not exist\n", i_id); -#endif -out: - vlc_mutex_unlock( &structure_lock ); - return obj; -} - -/** - **************************************************************************** +/***************************************************************************** * find a typed object and increment its refcount ***************************************************************************** * This function recursively looks for a given object type. i_mode can be one -- 2.39.2