2009/4/22 Michael Rueger <m.rueger@acm.org>:
Hi all,
in the Pharo inbox there is now Kernel-michael_rueger.renggli.304
integrated John's Mac menu fixes fixed problems with shift cursor fixed and cleaned up input sensor/fetcher installation
After loading the package you need to execute InputEventSensor installPollingEventSensorFramework
Hopefully this will fix all the recent issues around input events.
Thanks to all contributors and testers!
Michael, have you adressed the potential problems , which i mentioned: - protect a handlers collection from concurrent access - installing a new event fetcher should preserve the list of handlers, by taking them from the currently installed one - ?maybe? , when installing new event fetcher, use #become: to make sure anything which were pointed to old fetcher will now point to a new one. I don't have an updated version, but i think the InputEventHandler doesn't needs to hold a reference to the fetcher i.e. because it creates a reference cycle (fetcher has a handler in its handlers collection, and handler has a fetcher in its ivar). I'd recomment to rewrite register/unregister procedure to just: InputEventFetcher default [un]registerHandler: self and make sure , fetcher cares about possible handler duplicates (registering an already registered handler should be an no-op).
Michael
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
-- Best regards, Igor Stasenko AKA sig.