Issue 5101 in pharo: better, cleaner, more tests for SharedPool

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

Issue 5101 in pharo: better, cleaner, more tests for SharedPool

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

New issue 5101 by [hidden email]: better, cleaner, more tests for  
SharedPool
http://code.google.com/p/pharo/issues/detail?id=5101

Marcus is you can have a look. Now we even have tests for each of the  
methods!
and correct comments.

Attachments:
        BetterTestsForPoolUsers.4.cs  14.5 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 5101 in pharo: better, cleaner, more tests for SharedPool

pharo

Comment #1 on issue 5101 by marianopeck: better, cleaner, more tests for  
SharedPool
http://code.google.com/p/pharo/issues/detail?id=5101

I will try to take a look on friday



_______________________________________________
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 5101 in pharo: better, cleaner, more tests for SharedPool

pharo

Comment #2 on issue 5101 by [hidden email]: better, cleaner, more  
tests for SharedPool
http://code.google.com/p/pharo/issues/detail?id=5101

in 14258.

Now funnily when loading the class, since sharedPool are not put first in  
the cs, the definition of multiple
pool user went wrong and the system generated a 'private' shared pool (an  
empty dictionary and not a sharedPool). Gorgeous


_______________________________________________
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 5101 in pharo: better, cleaner, more tests for SharedPool

pharo
Updates:
        Status: Closed

Comment #3 on issue 5101 by [hidden email]: better, cleaner, more  
tests for SharedPool
http://code.google.com/p/pharo/issues/detail?id=5101

I hope that I will be able to save and load MCPackages....


_______________________________________________
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 5101 in pharo: better, cleaner, more tests for SharedPool

pharo

Comment #4 on issue 5101 by marianopeck: better, cleaner, more tests for  
SharedPool
http://code.google.com/p/pharo/issues/detail?id=5101

"Now funnily when loading the class, since sharedPool are not put first in  
the cs,"
In Fuel we found a similar problem. When we are serilizing the classPool,  
what should be first, that or the class itself?  because a global variable  
can be an instance of the same class :)   In your example, your sharedPool  
SharedPoolXXX can have a class side variable YYY which is an instance of  
SharedPoolXXX


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