Henry, No argument, your solution is simpler. However, we do not have announcements like this now, and it appears that the poor state of weak collections in Squeak/Pharo is standing in the way of a clean implementation. My vote is to tackle the weaklings, at least until they work well enough to support what we should have in this case. Otherwise, we kick the weak can down the road as has been done all along rather than fixing it, AND we end up with a sub-standard announcments-based framework. Let's do it right. We will then have an evolving/improving weak collection and something in the image that puts some stress on same. If I'm missing something, please enlighten me. Bill -----Original Message----- From: pharo-project-bounces@lists.gforge.inria.fr [mailto:pharo-project-bounces@lists.gforge.inria.fr] On Behalf Of Henrik Sperre Johansen Sent: Thursday, January 07, 2010 7:45 PM To: pharo-project@lists.gforge.inria.fr Subject: Re: [Pharo-project] Notifications of windows opening/closing/focus On 08.01.2010 01:32, Schwab,Wilhelm K wrote:
One should always be *able* to unsubscribe, but it should not be required.
Squeak's weak collections do not measure up to the competition. Please take a look at my 1/2/2010 post "RE: [Pharo-project] Can't debug in 1.1". I doubt SharedLookupTable could be called efficient, but it solved some problems I was having due to the current implementation. I welcome concrete examples of how horrible it is, and offer it up as a starting point or simple inspiration for something more useful.
Bill
I guess I should have quoted Gary's question of whether the approach would be hard to unregister instead. Originally, I (like you) thought not. After actually looking at the implementation, I do. What I posted is one way to "solve" this, weak subscriptions is another. On a scale of "what's harder to get right", I'd bet there's a larger chance of having 2 arg blocks implemented first. Cheers, Henry _______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project