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

Georg Köster georg.koester at gmail.com
Sat Jan 5 13:11:05 UTC 2008


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!

Georg

On Jan 5, 2008 1:47 PM, Alexander Lazarević <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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20080105/4f253434/attachment.htm


More information about the Squeak-dev mailing list