stream: Do not yield and then release. It is unuseful here.

Because there is no proper convention, there is no reason why the object couldn't be detached. We would need to vlc_object_find the input to be sure there is no problem. Let's assume for now that the input creates the access and is responsible for it.

diff --git a/src/input/stream.c b/src/input/stream.c
index 16ec8877185a094ddc5dfe8514f572deba177747..35b6454a03deb5514915e4740c3c6b7281662107 100644 (file)
--- a/src/input/stream.c
+++ b/src/input/stream.c
@@ -1687,7 +1687,6 @@ static int AReadStream( stream_t *s, void *p_read, int i_read )
         i_read = p_access->pf_read( p_access, p_read, i_read );
         if( p_input )
         {
-            vlc_object_yield( p_input );
             vlc_mutex_lock( &p_input->p->counters.counters_lock );
             stats_UpdateInteger( s, p_input->p->counters.p_read_bytes, i_read,
                              &i_total );
@@ -1695,7 +1694,6 @@ static int AReadStream( stream_t *s, void *p_read, int i_read )
                            (float)i_total, NULL );
             stats_UpdateInteger( s, p_input->p->counters.p_read_packets, 1, NULL );
             vlc_mutex_unlock( &p_input->p->counters.counters_lock );
-            vlc_object_release( p_input );
         }
         return i_read;
     }