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
|