Hi,

I see my first PR got integrated too fast. I've made a new one this morning that removes the guard and adds some comments in the code.

Also, answering the questions that arised in the PR, I believe this should have no effect on systems using a single window.
However, with multiple windowing systems, this (my patch) makes the consumption of events a cooperative task between the windowing systems.
For example, if we manage to have a SDL window open but SDL does not consume its events (because we don't call the event processing routine), then those events stay in the queue for ever.

I'm still not happy with the solution but I can deal with it right now, and it should not affect anybody else...

To make it more robust I've been trying to
 - Use a separate queue to store "alien" events and then repost them in the queue afterwards (https://developer.apple.com/documentation/appkit/nsapplication/1428710-postevent?language=objc).
    But I could not manage to make it work yet, and Cocoa's support/documentation does really NOT help here... :)
- Maybe we can just discard "very old" events?

But on the other side this solution is maybe simple enough to do the job.

Comments are welcome ^^
Cheers,
Guille

On Mon, Oct 22, 2018 at 3:16 PM Eliot Miranda <eliot.miranda@gmail.com> wrote:
 
Hi Guille,

On Oct 22, 2018, at 3:31 AM, Guillermo Polito <guillermopolito@gmail.com> wrote:

Hi all,

While playing with SDL on OSX we have noticed that the main window in the VM is randomly eating events that belong to other windows. This happens because we are consuming events regardless the window they come from.

The corresponding code is in the method pumpRunLoopEventSendAndSignal: in

I've playing around with making the VM consume only events that come from its own window or from no window (window=null).
This seems to work correctly, and I've had run into no issues in the last couple of hours.

I've also prepared a PR with the associated changes in here: https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/295.
I've put a guard so my code is only valid for PHAROVM builds, but I'd gladly remove the guards if people think this is a good move for all the community.

Yes please!

Thanks,
Guille


--

   

Guille Polito

Research Engineer

Centre de Recherche en Informatique, Signal et Automatique de Lille

CRIStAL - UMR 9189

French National Center for Scientific Research - http://www.cnrs.fr


Web: http://guillep.github.io

Phone: +33 06 52 70 66 13