> I tested that with your fixes, various category modifications (adding 
> new categories, changing categories, etc.) are now properly 
> tracked by 
> the current changeset.  Approved for 3.7beta.

That's great! 

Thanks very much.
Nathanael

> -----Original Message-----
> From: squeak-dev-bounces at lists.squeakfoundation.org 
> [mailto:squeak-dev-bounces at lists.squeakfoundation.org] On 
> Behalf Of Doug Way
> Sent: Donnerstag, 15. April 2004 18:44
> To: The general-purpose Squeak developers list
> Subject: Re: [KCP][FIX] 
> KCP-0220-ClassOrganizerFixAndCleanup.8.cs ([et][approved][for 
> 3.7beta] use Nathanael's new .14.cs)
> 
> 
> Nathanael Schärli wrote:
> 
> >Hi Doug, Stef, and Marcus
> >  
> >
> >>Doug wrote:
> >>
> >>I agree that it definitely looks like a bug and your fix looks good.
> >>Although I notice that your fix doesn't cover the case of a 
> >>new method 
> >>category being added. :-)  If you submit another fix for 
> >>this, go ahead 
> >>and post under a new subject, since it is separate from KCP220.
> >>    
> >>
> >
> >It is true that the little fix I posted does not cover the case when 
> >method categories are added. However, this is precisely what 
> is fixed 
> >by the changeset KCP220! This changeset addresses the fact 
> that there 
> >were many missing notifications in the ClassOrganizer (for 
> example when 
> >categories are added/removed).
> >
> 
> Ah, cool, I didn't test that with KCP220.  So then KCP220 definitely 
> fixes some things. ;-)
> 
> I tested that with your fixes, various category modifications (adding 
> new categories, changing categories, etc.) are now properly 
> tracked by 
> the current changeset.  Approved for 3.7beta.
> 
> - Doug
> 
> 
>