[KCP][FIX] KCP-0220-ClassOrganizerFixAndCleanup.8.cs
([et][approved][for 3.7beta] use Nathanael's new .14.cs)
Doug Way
dway at mailcan.com
Thu Apr 15 16:43:36 UTC 2004
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
More information about the Squeak-dev
mailing list
|