[vlc-devel] [RFC 06/10] input/demux: prepare input_DemuxNew for future functionality

Filip Roséen filip at atch.se
Mon Nov 28 17:56:57 CET 2016


Hi Remi,

On 2016-11-28 18:52, Rémi Denis-Courmont wrote:

> Nack. Demuxer code has no business messing with input thread. It´s
> even not the only use case for demux..

The alternative would be to move the stream-creation to outside the
relevant function, I have a patch for that, but it is more intrusive
than what is addressed in this patch (which is why I decided to push
this one instead of the other).

I do however agree with you, and I have for a long time been feeling
very anxious regarding the fact that we conditionally create the
input-stream chain inside `src/input/demux.c`.

Best Regards,\
Filip
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20161128/15ead2fd/attachment.html>


More information about the vlc-devel mailing list