winemac: Clean up fully-delivered events in queues which aren't being drained.

Some events get queued for all GUI-connected threads but are only processed by the first to dequeue them. Other threads which tend their event queue discard such already-processed events. However, some threads may be connected to the GUI but never tend their event queue. To prevent such threads from accumulating zombie events, the zombies are cleared each time a new event is queued.

winemac: Clean up fully-delivered events in queues which aren't being drained.
761092a2 · Ken Thomases · Alexandre Julliard · 9deb00bf · 761092a2
Commit 761092a2 authored Oct 17, 2013 by Ken Thomases Committed by Alexandre Julliard Oct 18, 2013
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

cocoa_event.m dlls/winemac.drv/cocoa_event.m +6 -0

No files found.
--- a/dlls/winemac.drv/cocoa_event.m
+++ b/dlls/winemac.drv/cocoa_event.m
@@ -171,10 +171,16 @@ static const OSType WineHotKeySignature = 'Wine';

    - (void) postEventObject:(MacDrvEvent*)event
    {
+        NSIndexSet* indexes;
        MacDrvEvent* lastEvent;

        [eventsLock lock];

+        indexes = [events indexesOfObjectsPassingTest:^BOOL(id obj, NSUInteger idx, BOOL *stop){
+            return ((MacDrvEvent*)obj)->event->deliver <= 0;
+        }];
+        [events removeObjectsAtIndexes:indexes];
+
        if ((event->event->type == MOUSE_MOVED ||
             event->event->type == MOUSE_MOVED_ABSOLUTE) &&
            event->event->deliver == INT_MAX &&