We don't need to detach. Since we're in the parent's destructor, it's already been...

diff --git a/src/video_output/video_output.c b/src/video_output/video_output.c
index 710d86093a32e9df3b3883ce623d0279c980843b..eb64a3e7be9b07b4e40500660a86ba445a2032b2 100644 (file)
--- a/src/video_output/video_output.c
+++ b/src/video_output/video_output.c
@@ -1223,7 +1223,6 @@ static void EndThread( vout_thread_t *p_vout )
     if( !p_vout->b_direct )
     {
         module_Unneed( p_vout->p_chroma, p_vout->p_chroma->p_module );
-        vlc_object_detach( p_vout->p_chroma );
         vlc_object_release( p_vout->p_chroma );
         p_vout->p_chroma = NULL;
     }

diff --git a/src/video_output/vout_subpictures.c b/src/video_output/vout_subpictures.c
index 14dd76dbd9457feedb5fa94ff9914f5c3ccd9995..70e81c2af2b40db3e8a3649fbb4b34beca693c9c 100644 (file)
--- a/src/video_output/vout_subpictures.c
+++ b/src/video_output/vout_subpictures.c
@@ -150,8 +150,6 @@ void spu_Destroy( spu_t *p_spu )
 {
     int i_index;
 
-    vlc_object_detach( p_spu );
-
     /* Destroy all remaining subpictures */
     for( i_index = 0; i_index < VOUT_MAX_SUBPICTURES; i_index++ )
     {