Thanks for the explanation. I'll commit that one as well, then.

On 10/18/05, Andreas Raab <andreas.raab at gmx.de> wrote:
> Cees De Groot wrote:
> > It still begs the question what the patch does ;-)
>
> That particular patch abstracts away SyntaxError, which despite its name
> is not an exception but a (rather ugly) subclass of StringHolder which
> is used in a fairly weird interaction with SyntaxErrorNotification
> (being the actual exception being raised). Because SyntaxError works
> directly with Debugger it is necessary to remove the references to it if
> you want to unload the tools package cleanly. Finally, a SyntaxError is
> actually being created when no requestor is given and much of the code
> in parser which deals with that looks like "requestor notNil and:[
> (requestor isKindOf: SyntaxError) not]". It's just very ugly code.
>
> Cheers,
>    - Andreas
>
>
>