mirror/mpv
0
0
Fork 0
mirror of https://github.com/mpv-player/mpv.git synced 2024-09-20 12:02:23 +02:00
Code

libmpv: cocoa: fix view leak on uninit

Browse Source 
The code was lacking a -removeFromSuperview call which resulted in a leak on
our part if the parent view in the client was not released.
This commit is contained in:
Stefano Pigozzi 2014-10-27 19:10:52 +01:00
parent 58effd3fec
commit 55396ee554
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
video/out/cocoa_common.m
View File

@ -192,9 +192,9 @@ void vo_cocoa_uninit(struct vo *vo)
enable_power_management(vo);
cocoa_rm_fs_screen_profile_observer(vo);
[s->video release];
[s->view release];
[s->window release];
[s->gl_ctx release];
[s->view removeFromSuperview];
if (s->window) [s->window release];
});
}
@ -310,7 +310,7 @@ static void create_ui(struct vo *vo, struct mp_rect *win, int geo_flags)
struct vo_cocoa_state *s = vo->cocoa;
struct mp_vo_opts *opts = vo->opts;
MpvCocoaAdapter *adapter = [[[MpvCocoaAdapter alloc] init] autorelease];
MpvCocoaAdapter *adapter = [[MpvCocoaAdapter alloc] init];
adapter.vout = vo;
NSView *parent;
@ -327,6 +327,7 @@ static void create_ui(struct vo *vo, struct mp_rect *win, int geo_flags)
view.adapter = adapter;
s->view = view;
[parent addSubview:s->view];
[s->view release];
// insert ourselves as the next key view so that clients can give key
// focus to the mpv view by calling -[NSWindow selectNextKeyView:]
@ -345,6 +346,7 @@ static void create_ui(struct vo *vo, struct mp_rect *win, int geo_flags)
[s->view addSubview:s->video];
[s->gl_ctx setView:s->video];
[s->video release];
s->video.adapter = adapter;