]> git.sesse.net Git - vlc/commitdiff
Remove vlc_threadobj (did not work properly anymore)
authorRémi Denis-Courmont <rdenis@simphalempin.com>
Sat, 30 Aug 2008 07:28:26 +0000 (10:28 +0300)
committerRémi Denis-Courmont <rdenis@simphalempin.com>
Sat, 30 Aug 2008 07:29:01 +0000 (10:29 +0300)
src/libvlc.h
src/misc/objects.c
src/misc/threads.c

index b934f0addb40f68e8b205d0c34cf66f356255599..d110a7642757aafcc02fcad5f41ae3797a4e3668 100644 (file)
@@ -45,7 +45,6 @@ void system_End       ( libvlc_int_t * );
  */
 int vlc_threads_init( void );
 void vlc_threads_end( void );
-vlc_object_t *vlc_threadobj (void);
 
 /* Hopefully, no need to export this. There is a new thread API instead. */
 void vlc_thread_cancel (vlc_object_t *);
index 6244439426413f0c3401a272cbae5cd6907c479d..58140dd3c8bf8e82ec4f4c02b9d0e81b821c1682 100644 (file)
@@ -594,17 +594,9 @@ void * vlc_object_get( int i_id )
     libvlc_global_data_t *p_libvlc_global = vlc_global();
     vlc_object_t *obj = NULL;
 #ifndef NDEBUG
-    vlc_object_t *caller = vlc_threadobj ();
-
-    if (caller)
-        msg_Dbg (caller, "uses deprecated vlc_object_get(%d)", i_id);
-    else
-    {
-        int canc = vlc_savecancel ();
-        fprintf (stderr, "main thread uses deprecated vlc_object_get(%d)\n",
-                 i_id);
-        vlc_restorecancel (canc);
-    }
+    int canc = vlc_savecancel ();
+    fprintf (stderr, "Use of deprecated vlc_object_get(%d)\n", i_id);
+    vlc_restorecancel (canc);
 #endif
     vlc_mutex_lock( &structure_lock );
 
@@ -620,10 +612,7 @@ void * vlc_object_get( int i_id )
     }
     obj = NULL;
 #ifndef NDEBUG
-    if (caller)
-        msg_Warn (caller, "wants non-existing object %d", i_id);
-    else
-        fprintf (stderr, "main thread wants non-existing object %d\n", i_id);
+    fprintf (stderr, "Object %d does not exist\n", i_id);
 #endif
 out:
     vlc_mutex_unlock( &structure_lock );
index 4480e41de190d365e78bae6fb87d4aacec52fb7a..133c695471c0bfbb78fc9f59e41e5d57cf7e3a61 100644 (file)
@@ -66,18 +66,6 @@ libvlc_global_data_t *vlc_global( void )
     return p_root;
 }
 
-#ifndef NDEBUG
-/**
- * Object running the current thread
- */
-static vlc_threadvar_t thread_object_key;
-
-vlc_object_t *vlc_threadobj (void)
-{
-    return vlc_threadvar_get (&thread_object_key);
-}
-#endif
-
 vlc_threadvar_t msg_context_global_key;
 
 #if defined(LIBVLC_USE_PTHREAD)
@@ -190,9 +178,6 @@ int vlc_threads_init( void )
         }
 
         /* We should be safe now. Do all the initialization stuff we want. */
-#ifndef NDEBUG
-        vlc_threadvar_create( &thread_object_key, NULL );
-#endif
         vlc_threadvar_create( &msg_context_global_key, msg_StackDestroy );
 #ifndef LIBVLC_USE_PTHREAD_CANCEL
         vlc_threadvar_create( &cancel_key, free );
@@ -230,9 +215,6 @@ void vlc_threads_end( void )
         vlc_threadvar_delete( &cancel_key );
 #endif
         vlc_threadvar_delete( &msg_context_global_key );
-#ifndef NDEBUG
-        vlc_threadvar_delete( &thread_object_key );
-#endif
     }
     i_initializations--;
 
@@ -661,9 +643,6 @@ static void *thread_entry (void *data)
     void *(*func) (vlc_object_t *) = ((struct vlc_thread_boot *)data)->entry;
 
     free (data);
-#ifndef NDEBUG
-    vlc_threadvar_set (&thread_object_key, obj);
-#endif
     msg_Dbg (obj, "thread started");
     func (obj);
     msg_Dbg (obj, "thread ended");