[Seaside] Able to hang the Squeak UI
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!
> Lukas Renggli
> Seaside mailing list
> Seaside at lists.squeakfoundation.org
More information about the Seaside