[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