cocoa: fix 'uninitalized' access

I think this doesn't make a difference, since in Objective-C nil responds as
a NullObject, but better not depend on this crappy language feature.

1 files changed, 7 insertions, 9 deletions

diff --git a/video/out/cocoa_common.m b/video/out/cocoa_common.m
index 7eabad53ba..d704706501 100644
--- a/video/out/cocoa_common.m
+++ b/video/out/cocoa_common.m
@@ -184,15 +184,13 @@ void vo_cocoa_uninit(struct vo *vo)
 
         [s->video release];
 
-        // XXX: It looks like there are some circular retain cycles for the
-        // video view / video window that cause them to not be deallocated,
-        // This is a workaround to make the fullscreen window be released,
-        // but the retain cycle problems should be investigated.
-        if ([s->view isInFullScreenMode])
-            [[s->view window] release];
-
-        [s->window release];
-        s->window = nil;
+        if (!s->embedded) {
+            if ([s->view isInFullScreenMode])
+                [[s->view window] release];
+
+            [s->window release];
+            s->window = nil;
+        }
     });
 }