[squeak-dev] Solving termination of critical sections in the context of priority inversion was: SemaphoreTest fails in trunk, is a fix needed for the 5.2 release?

Fri Jul 27 18:44:04 UTC 2018

> On 27.07.2018, at 18:01, Eliot Miranda <eliot.miranda at gmail.com> wrote:
> 
> Hi Tobias,
[…]
> 
>> Just one nit. Yes we know the method sends #ensure:, but should the code below also be run for other methods that are marked by primitive 198?
>> (eg, #ifCurtailed:, as send in Semaphore>>critical:ifCurtailed:).
>> I am just asking :) checking for the prim might be a tiny tad more robust.
> 
> Doh!  Much better.  
> 
>    unwindProtectPrimitiveNumber := 198.
>    ...
>    suspendedContext sender method primitive = unwindProtectPrimitiveNumber ifTrue:
> 
> or implement isUnwindProtect in Context.
> 
> Thanks!!

No biggie ;)
	-t