Simple lock up with delay + semaphore - not fixed with 0006576

Andreas Raab andreas.raab at gmx.de
Sat Jan 5 13:15:10 UTC 2008


Georg Köster wrote:
> Hi Alex,
> 
> thanks for the note - but in my case I'm on the Croquet image and on the 
> 3.8.18beta1U and my code looks like it should!

So then what exactly is your problem? Like I was saying earlier, the 
code you posted appears to work fine in the Croquet SDK image.

Cheers,
   - Andreas

> 
> Georg
> 
> On Jan 5, 2008 1:47 PM, Alexander Lazarević < Alexander at lazarevic.de 
> <mailto:Alexander at lazarevic.de>> wrote:
> 
>     Andreas Raab schrieb:
>      > Georg Köster wrote:
>      >> This shows the use case. Shorter even:
>      >> |s|
>      >> s := Semaphore new.
>      >> Delay timeoutSemaphore: s afterMSecs: 1.
>      >> s wait.
>      >> Transcript show: 'doesnt get here - not printed'; cr.
>      >>
>      >> I hope I'm doing something wrong here!
>      >
>      > You must be. This works fine in a Croquet SDK image.
>     Unfortunately it seems that a ChangeSet of mine addressing [1]
>     introduced this bug. I guess it got included in 3.9 without anyone else
>     looking at it. As I mention in [2] the Delay class side method
>     should read:
> 
>     timeoutSemaphore: aSemaphore afterMSecs: anInteger
>       "...comment..."
> 
>        ^ (self new setDelay: anInteger forSemaphore: aSemaphore) schedule
> 
>     I guess trying to refactor too much made me change the method. I'm sorry
>     that I did not double check it more carefully in the first place.
> 
>     Alex
> 
>     [1] http://bugs.squeak.org/view.php?id=854
>     [2] http://bugs.squeak.org/view.php?id=6834
>     <http://bugs.squeak.org/view.php?id=6834>
> 
> 
> 
> ------------------------------------------------------------------------
> 
> 




More information about the Squeak-dev mailing list