Last integration went too fast :)
This PR does some cleanups: - remove the PharoVM guards, - added comments
Answering the questions from the other thread: this should have no effect on systems using a single window. However, with multiple windowing systems, this makes the consumption of events a cooperative task. If let's say SDL does not consume its events (because we don't call the event processing routine), then those events stay in the queue.
To make it more robust I've been trying to use a separate queue to store alien events and repost them in the queue afterwads, but I could not manage to make it work. And Cocoa's support/documentation does not help here... You can view, comment on, or merge this pull request online at:
https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/296
-- Commit Summary --
* Added comments and removed PharoVM guards
-- File Changes --
M platforms/iOS/vm/OSX/sqSqueakOSXApplication+events.m (31)
-- Patch Links --
https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/296.patch https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/296.diff
eliotmiranda approved this pull request.
I'd rather have this merged and in use than not being tested. We can fix any breakages later. As long as the simple case works we can then test the more complex cases more easily bu=y having this in the default VM.
Merged #296 into Cog.
vm-dev@lists.squeakfoundation.org