[vlc-devel] [PATCH 1/2] stream: add stream_DemuxAlive

Julian Scheel julian at jusst.de
Mon Apr 20 16:52:25 CEST 2015


On 20.04.2015 15:25, Rémi Denis-Courmont wrote:
> Le 2015-04-20 15:33, Julian Scheel a écrit :
>> This will cause the access_demux to eat all system memory. Shouldn't
>> we have a way out there?
>
> You should not use the current stream_Demux in that case. That's all I
> can say.

Which brings us back to the initial patch by Francois...
Just as francois I don't really see why it's racy - it's just allowing 
to read an atomic state variable from outside. But I surely miss 
something due to lack of knowledge about the stream_Demux internals?

What about just having sys->finished? Atomic init to false on creation 
and atomic write to true right after demux_Delete in DStreamThread?

-Julian




More information about the vlc-devel mailing list