My 2cts is for some event (and I think *modified is part of them), you got the event twice: before and after. So basically, one of the two events is wrong. Mariano, you should try to put an halt, and see :) Ben On Jun 16, 2012, at 10:26 PM, Guillermo Polito wrote:
Yeah, but logging superfluous changes does not really make a change, does it?
On Sat, Jun 16, 2012 at 10:04 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote: may be because the source changes was listening to the event. And the log file needed to log any thing like doits and other.
On Jun 16, 2012, at 9:14 PM, Mariano Martinez Peck wrote:
Hi guys. ChangeSet does at some point:
anEvent itemKind = SystemChangeNotifier classKind ifTrue: [ ... anEvent isModified ifTrue: [anEvent anyChanges ifTrue: [self changeClass: anEvent item from: anEvent oldItem]]. anEven
So...I don't understand the #anyChanges. We received an event saying that a class was modified, but it may not have any changes? if true, why was the event sent? Ok...let's say that there could be several reasons that a class was modified but only some of them make sense to change set. I can imagine people wanted to avoid writing stuff to .changes that didn't actullay represente a change?
If this is the case, then #anyChanges looks like a bad name for me.
Now, do we still need this mesage? can there be ClassModified events without changes? thanks
-- Mariano http://marianopeck.wordpress.com