[Vm-dev] [OpenSmalltalk/opensmalltalk-vm] Added comments and removed PharoVM guards for windowing support (#296)

Guille Polito notifications at github.com
Tue Oct 23 08:58:15 UTC 2018


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

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/296
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20181023/0a83220a/attachment-0001.html>


More information about the Vm-dev mailing list