Re: Issue 2255 in pharo: deprecated Symbol>>#flushCache

Updates:
        Status: FixReviewNeeded
        Labels: Type-Cleanup

Comment #1 on issue 2255 by [hidden email]: deprecated  
Symbol>>#flushCache
http://code.google.com/p/pharo/issues/detail?id=2255

if we want to do that, fix attached...

Attachments:
        FlushCache.1.cs  2.2 KB


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

Updates:
        Cc: [hidden email] [hidden email]

Comment #2 on issue 2255 by marianopeck: deprecated Symbol>>#flushCache
http://code.google.com/p/pharo/issues/detail?id=2255

Hi Marcus. I am not sure if we want this. As far as I understand, they are  
differnt primitives. One searches in the cache by CM and the other one by  
selector. Sometimes you want to flush cache but you only have the selector?




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

Comment #3 on issue 2255 by [hidden email]: deprecated  
Symbol>>#flushCache
http://code.google.com/p/pharo/issues/detail?id=2255

I agree with Mariano. The issue was that calling flush on CM and symbol was  
not the same. Eliot fixed this to be homogeneous. Now, having the  
possibility of flushing from a symbol is sometimes important.


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

Comment #4 on issue 2255 by [hidden email]: deprecated  
Symbol>>#flushCache
http://code.google.com/p/pharo/issues/detail?id=2255

ok, second version.

-> change comments
-> avoid duplicate flushCache when replacing methods
-> avoid duplicate flushCache when deleting a method
-> avoid flush cache when just adding a new method

Attachments:
        FlushCache2.1.cs  1.9 KB


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