winemac: Wrap performing requests from background threads in an autorelease pool.

Cocoa manages an autorelease pool on the main thread, but it only drains it when it processes an event. Our requests come through a run loop source, which doesn't count as an event. So, autoreleased objects can accumulate when the app is not being interacted with. Signed-off-by: Ken Thomases <ken@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winemac: Wrap performing requests from background threads in an autorelease pool.
e5c12089 · Ken Thomases · Alexandre Julliard · 58719f60 · e5c12089
Commit e5c12089 authored Feb 17, 2016 by Ken Thomases Committed by Alexandre Julliard Feb 18, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 0 deletions

cocoa_app.m dlls/winemac.drv/cocoa_app.m +5 -0

No files found.
--- a/dlls/winemac.drv/cocoa_app.m
+++ b/dlls/winemac.drv/cocoa_app.m
@@ -2295,6 +2295,7 @@ static void PerformRequest(void *info)

    for (;;)
    {
+        NSAutoreleasePool* pool = [[NSAutoreleasePool alloc] init];
        __block dispatch_block_t block;

        dispatch_sync(controller->requestsManipQueue, ^{
@@ -2308,10 +2309,14 @@ static void PerformRequest(void *info)
        });

        if (!block)
+        {
+            [pool release];
            break;
+        }

        block();
        [block release];
+        [pool release];
    }
 }