[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