Ok. Deprecated it, but in a way that makes the right code run, not the
code that was called before. Can you review this?

It seems removeClassFromSystem:logged: also removes the class from
various things other than the SystemDictionary - the startup and
shutdown lists, for example. Therefore I ended up naming it
forgetClass:logged:, which is hopefully vague enough that people will
read the method before trying to use it directly.

Daniel

"David T. Lewis" <lewis at mail.msen.com> wrote:
> On Sun, Sep 14, 2003 at 08:00:34PM +0300, Daniel Vainsencher wrote:
> > Yes, I looked, and Roels change is for a rename method which is
> > similarly split, not for removal. So I propose:
> > removeClassFromSystem:logged: -> removeBindingForClass:logged:
> > 
> > Any objections alternatives? if not, I'll send in a changeset.
> 
> Sounds reasonable enough.  An alternative might be to call it
> #removeClassFromSystemDictionary:logged:, but I think your suggestion
> sounds fine.
> 
> The original method should be deprecated (not just renamed), because
> some packages on SqueakMap may be relying it in for loading change
> sets. Chronology currently does, and maybe others as well, so these
> should be permitted to fail gracefully if the method is renamed.
> 
> Dave