[Seaside] Able to hang the Squeak UI

Philippe Marschall philippe.marschall at gmail.com
Tue Jul 3 20:07:00 UTC 2007


2007/7/3, Lukas Renggli <renggli at gmail.com>:
> > What did you find?  What did you fix?
>
> The variable semaphore in WAProcessMonitor>>#critical:ifError: has to
> be an instance-variable, even so it is only used within this method.
>
> This was changed some time ago, as it seemed to be more natural to
> make it a temp-var. However this introduced a subtle regression with
> continuations causing the lock not to be released in some situations
> with the debugger. The issue is now documented in the method itself
> and should not occur anymore!

This class should have a comment 'here be dragons'. I don't know how
many times I found out it didn't work the way I thought. I surely
wouldn't miss it if we could get rid of it. Great work!

Philippe

> Lukas
>
> --
> Lukas Renggli
> http://www.lukas-renggli.ch
> _______________________________________________
> Seaside mailing list
> Seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>


More information about the Seaside mailing list