Issue 4922 in pharo: duplicate logic in SystemChangeNotifier

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 4922 in pharo: duplicate logic in SystemChangeNotifier

pharo
Status: FixToInclude
Owner: [hidden email]
Labels: Milestone-1.4

New issue 4922 by [hidden email]: duplicate logic in  
SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=4922

This one is better.

SystemChangeNotifier>>methodChangedFrom: oldMethod to: newMethod selector:  
aSymbol inClass: aClass
        ^ self methodChangedFrom: oldMethod to: newMethod selector: aSymbol  
inClass: aClass requestor: nil


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 4922 in pharo: duplicate logic in SystemChangeNotifier

pharo
Updates:
        Cc: [hidden email]

Comment #1 on issue 4922 by [hidden email]: duplicate logic in  
SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=4922

And this is strange that ModifiedEvent is notified instead of  
ModifiedMethodEvent


Attachments:
        ChangeNotifierChanges.1.cs  733 bytes


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 4922 in pharo: duplicate logic in SystemChangeNotifier

pharo

Comment #2 on issue 4922 by [hidden email]: duplicate logic in  
SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=4922

Two different events were raised when a method was modified
ModifiedEvent and ModifiedMethodEvent.

This changes makes sure that only one is raised.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 4922 in pharo: duplicate logic in SystemChangeNotifier

pharo

Comment #3 on issue 4922 by [hidden email]: duplicate logic in  
SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=4922

(No comment was entered for this change.)

Attachments:
        ModifiedMethod.1.cs  741 bytes


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 4922 in pharo: duplicate logic in SystemChangeNotifier

pharo
Updates:
        Status: Closed

Comment #4 on issue 4922 by [hidden email]: duplicate logic in  
SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=4922

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker