[vlc-devel] [PATCH] test: add thread test

Romain Vimont rom1v at videolabs.io
Fri Mar 20 19:28:21 CET 2020


On 3/20/20 6:26 PM, Romain Vimont wrote:
> 
> 
> On 3/20/20 6:13 PM, Rémi Denis-Courmont wrote:
>> Le perjantaina 20. maaliskuuta 2020, 18.31.02 EET Alexandre Janniaux a 
>> écrit :
>>> Hi,
>>>
>>> I don't understand the issue,
>>
>> The test thread signals itself. Then everything locks dead.
>>
> 
> Oh, right. Good catch. I totally missed it.
> 
> Replacing vlc_cond_signal() by vlc_cond_broadcast() should fix the issue 
> then :)

Sorry, thinking about it, in fact I don't understand the problem.

A thread calling cond_signal() may not "signal itself" since it's not 
cond_wait()ing. :/


More information about the vlc-devel mailing list