Issue 7165 in pharo: CleanUp MethodChangeRecord

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

Issue 7165 in pharo: CleanUp MethodChangeRecord

pharo
Status: FixReviewNeeded
Owner: [hidden email]
Labels: Type-Cleanup Milestone-2.0

New issue 7165 by [hidden email]: CleanUp MethodChangeRecord
http://code.google.com/p/pharo/issues/detail?id=7165

iVar currentMethod is not used, allows to clean up a little:

MethodChangeRecord removeSelector: #noteNewMethod:!
ClassChangeRecord removeSelector: #initFor:revertable:!
ClassChangeRecord removeSelector: #zapHistory!
ChangeSet removeSelector: #zapHistory!

+ rewrites of method to remove callers


Attachments:
        CleanMethodChangeRecord.cs  2.1 KB


_______________________________________________
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 7165 in pharo: CleanUp MethodChangeRecord

pharo
Updates:
        Status: FixToInclude

Comment #1 on issue 7165 by [hidden email]: CleanUp  
MethodChangeRecord
http://code.google.com/p/pharo/issues/detail?id=7165

(No comment was entered for this change.)


_______________________________________________
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 7165 in pharo: CleanUp MethodChangeRecord

pharo
Updates:
        Status: Integrated

Comment #2 on issue 7165 by [hidden email]: CleanUp  
MethodChangeRecord
http://code.google.com/p/pharo/issues/detail?id=7165

in 2.0 450


_______________________________________________
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 7165 in pharo: CleanUp MethodChangeRecord

pharo
Updates:
        Status: FixToInclude

Comment #3 on issue 7165 by [hidden email]: CleanUp  
MethodChangeRecord
http://code.google.com/p/pharo/issues/detail?id=7165

I only added the new method in 450

the cs can now be loaded


_______________________________________________
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 7165 in pharo: CleanUp MethodChangeRecord

pharo
Updates:
        Status: Integrated

Comment #4 on issue 7165 by [hidden email]: CleanUp  
MethodChangeRecord
http://code.google.com/p/pharo/issues/detail?id=7165

in 2.0 453


_______________________________________________
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 7165 in pharo: CleanUp MethodChangeRecord

pharo
Updates:
        Status: FixToInclude

Comment #5 on issue 7165 by [hidden email]: CleanUp  
MethodChangeRecord
http://code.google.com/p/pharo/issues/detail?id=7165

another fix to make it loadable in 453


_______________________________________________
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 7165 in pharo: CleanUp MethodChangeRecord

pharo
Updates:
        Status: Integrated

Comment #6 on issue 7165 by [hidden email]: CleanUp  
MethodChangeRecord
http://code.google.com/p/pharo/issues/detail?id=7165

in 2.0 454


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