Issue 7380 in pharo: Replace MCDefinition's Instances class variable with a class instance variable

Status: Accepted
Owner: ----
Labels: Milestone-2.0 Type-Cleanup

New issue 7380 by [hidden email]: Replace MCDefinition's Instances  
class variable with a class instance variable
http://code.google.com/p/pharo/issues/detail?id=7380

I'm cherry-picking from Monticello versions in  
http://source.squeak.org/trunk

This one seems good:


Name: Monticello-ul.375
Author: ul
Time: 28 February 2010, 2:58:21.945 pm
UUID: 2e3b9ffd-b728-364b-84c2-1f189c89fcdf
Ancestors: Monticello-ul.374

- replaced MCDefinition's Instances class variable with a class instance  
variable. Smaller WeakSets are better than a large one.



Slice comming...


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

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 7380 by [hidden email]: Replace MCDefinition's  
Instances class variable with a class instance variable
http://code.google.com/p/pharo/issues/detail?id=7380

Slice in the inbox:


Name:  
SLICE-Issue-7380-Replace-MCDefinitions-Instances-class-variable-with-a-class-instance-variable-MartinDias.1
Author: MartinDias
Time: 31 January 2013, 6:27:09.271 pm
UUID: 3144be9e-ea03-4899-83fa-99cf9c5b92c8
Ancestors:
Dependencies: Monticello-MartinDias.749

I'm cherry-picking from Monticello versions in  
http://source.squeak.org/trunk

This one seems good:


Name: Monticello-ul.375
Author: ul
Time: 28 February 2010, 2:58:21.945 pm
UUID: 2e3b9ffd-b728-364b-84c2-1f189c89fcdf
Ancestors: Monticello-ul.374

- replaced MCDefinition's Instances class variable with a class instance  
variable. Smaller WeakSets are better than a large one.



No idea if there was an specific test for this, but at least I run (in  
20501) all Tests-Monticello and Ring-Tests-Monticello, and it's ok: the  
same two tests have a progress bar ZeroDivide (I don't know if there is a  
issue report for these two guys).


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

Updates:
        Labels: -Milestone-2.0 Milestone-3.0

Comment #2 on issue 7380 by [hidden email]: Replace MCDefinition's  
Instances class variable with a class instance variable
http://code.google.com/p/pharo/issues/detail?id=7380

I think we should stop changing 2.0... lets move it to 3.0


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