[vlc-devel] [PATCH 2/8] sout_mux/core: handle drain
remi at remlab.net
Tue Jan 30 17:25:20 CET 2018
Le tiistaina 30. tammikuuta 2018, 10.22.57 EET Thomas Guillem a écrit :
> On Mon, Jan 29, 2018, at 19:33, Rémi Denis-Courmont wrote:
> > Le maanantaina 29. tammikuuta 2018, 19.25.50 EET Thomas Guillem a écrit :
> > > Like for decoder_t, a drain is requested when sending/writing a NULL
> > > block.
> > >
> > > Muxers should drain themselves when they receive the last pf_delstream
> > > callback (call the new sout_MuxIsDrained() helper to know when it
> > > happens).
> > I do not agree there. If it is possible to add and remove ES's
> > dynamically, it should be possible to remove all of them, then add new
> > ones. This is neither sufficient nor necessary a condition to drain the
> > muxer.
> There should be 2 muxers categories: one like TS that can handle ES
...and draining on last remaing ES is wrong for those muxers.
> And others like mkv that can't cope with streams change when
> they are already started (see the ugly sout-mux-caching option).
...and draining on last remaining ES is pointless for those muxers.
Muxers can and should drain on deletion, regardless of the "category".
More information about the vlc-devel