[vlc-devel] [PATCH] lib: media: reset parsed_status on new parse request

Remi Denis-Courmont remi at remlab.net
Wed Oct 23 19:11:53 CEST 2019


Le 2019-10-22 18:03, Thomas Guillem a écrit :
> On Tue, Oct 22, 2019, at 13:28, Rémi Denis-Courmont wrote:
> 
>> Hi,
>> 
>> Just a few days ago, Hugo made the case that media can be used in
>> multiple threads. Now, this seems to be doing the exact opposite,
>> propose a design that won't work if multiple threads use the same
>> media (and try to parse it).
> 
> If a second thread try to parse the media that is already parsing, it
> will return an error.
> 
> The media can be read from any threads, but only one thread can parse
> it now.

That makes no sense to me.

Either object can be *meaningfully* used in multiple threads or it 
can't. Reading something that can change asynchronously is not 
meaningful. Either this patch is wrong, or the other one is.

-- 
Rémi Denis-Courmont


More information about the vlc-devel mailing list