Changeset 81e3de1b3a1e02f5724d7add82340a9536a3e875

Show
Ignore:
Timestamp:
03/30/08 01:33:39 (5 months ago)
Author:
Pierre d'Herbemont <pdherbemont@free.fr>
git-committer:
Pierre d'Herbemont <pdherbemont@free.fr> 1206837219 +0100
git-parent:

[f8b8b4bfeb91ea2e4d55a68dd9fe7762e3a702d1]

git-author:
Pierre d'Herbemont <pdherbemont@free.fr> 1206837219 +0100
Message:

video_output: Don't keep the input around for a long time in RunThread?.

Else there is a circular dependency between the vout that yield() the input, where the vout is also an input child. This should fix properly the media_instance() tests.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • src/video_output/video_output.c

    radee2d2 r81e3de1  
    781781        current_date = mdate(); 
    782782 
    783         if( p_input && p_input->b_die ) 
    784         { 
    785             vlc_object_release( p_input ); 
    786             p_input = NULL; 
    787         } 
    788  
    789783        i_loops++; 
    790784            if( !p_input ) 
     
    803797                i_displayed = i_lost = 0; 
    804798                vlc_mutex_unlock( &p_input->p->counters.counters_lock ); 
     799                vlc_object_release( p_input ); 
     800                p_input = NULL; 
    805801            } 
    806802#if 0 
     
    10591055            p_subpic = spu_SortSubpictures( p_vout->p_spu, display_date, 
    10601056            p_input ? var_GetBool( p_input, "state" ) == PAUSE_S : VLC_FALSE ); 
     1057            if( p_input ) 
     1058                vlc_object_release( p_input ); 
     1059            p_input = NULL; 
    10611060        } 
    10621061