[vlc-devel] [PATCH 2/3] stream_Demux: handle slave demuxes

[Rémi Denis-Courmont]

Le 2015-09-14 16:22, Francois Cartegnie a écrit :
> Le 14/09/2015 14:36, Rémi Denis-Courmont a écrit :
>>
>>> +        demux_Control( p_demux, DEMUX_SET_NEXT_DEMUX_TIME,
>>> p_sys->i_next_demux_time );
>>> +        p_sys->i_next_demux_time += CLOCK_FREQ / 4;
>>
>> That can't work in any meaingful way. DEMUX_SET_NEXT_DEMUX_TIME is
>> supposed to make the next demux_Demux() wait for the time to be 
>> reached.
>> You have no ways to wait here, so this is a useless no-op.
>
> Subtitle demux does return data it owns up to NEXT_DEMUX_TIME, on 
> next
> demux() call.

Yes. That's what I wrote above, you know.

And that's also why it can't work from the stream demux thread.

-- 
Rémi Denis-Courmont
http://www.remlab.net/