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

Francois Cartegnie fcvlcdev at free.fr
Mon Sep 14 15:22:48 CEST 2015


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.
Placing it in a Thread will never return anything if no ones increments
that barrier.


Francois





More information about the vlc-devel mailing list