vlc | branch: master | David Fuhrmann <dfuhrm...@videolan.org> | Mon Oct 30 
12:11:22 2017 +0100| [905709d6e19cb032a9c646a6c7fc18282b021d16] | committer: 
David Fuhrmann

macosx: Use unsafe_unretained instead of weak for controller vars

weak is not supported for NSWindowController instances on Lion, using
it crashes VLC on 10.7.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=905709d6e19cb032a9c646a6c7fc18282b021d16
---

 modules/gui/macosx/VLCAudioEffectsWindowController.m   | 8 ++++----
 modules/gui/macosx/VLCConvertAndSaveWindowController.m | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/modules/gui/macosx/VLCAudioEffectsWindowController.m 
b/modules/gui/macosx/VLCAudioEffectsWindowController.m
index e337395e2c..89f7744453 100644
--- a/modules/gui/macosx/VLCAudioEffectsWindowController.m
+++ b/modules/gui/macosx/VLCAudioEffectsWindowController.m
@@ -479,7 +479,7 @@
     [_textfieldPanel setCancelButtonString:_NS("Cancel")];
     [_textfieldPanel setOkButtonString:_NS("Save")];
 
-    __weak typeof(self) _self = self;
+    __unsafe_unretained typeof(self) _self = self;
     [_textfieldPanel runModalForWindow:self.window 
completionHandler:^(NSInteger returnCode, NSString *resultingText) {
 
         NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults];
@@ -533,7 +533,7 @@
     [_popupPanel setCancelButtonString:_NS("Cancel")];
     [_popupPanel setPopupButtonContent:[[NSUserDefaults standardUserDefaults] 
objectForKey:@"AudioEffectProfileNames"]];
 
-    __weak typeof(self) _self = self;
+    __unsafe_unretained typeof(self) _self = self;
     [_popupPanel runModalForWindow:self.window completionHandler:^(NSInteger 
returnCode, NSInteger selectedIndex) {
 
         NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults];
@@ -788,7 +788,7 @@ static bool GetEqualizerStatus(intf_thread_t *p_custom_intf,
     [_textfieldPanel setCancelButtonString:_NS("Cancel")];
     [_textfieldPanel setOkButtonString:_NS("Save")];
 
-    __weak typeof(self) _self = self;
+    __unsafe_unretained typeof(self) _self = self;
     [_textfieldPanel runModalForWindow:self.window 
completionHandler:^(NSInteger returnCode, NSString *resultingText) {
         NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults];
 
@@ -834,7 +834,7 @@ static bool GetEqualizerStatus(intf_thread_t *p_custom_intf,
     [_popupPanel setCancelButtonString:_NS("Cancel")];
     [_popupPanel setPopupButtonContent:[[NSUserDefaults standardUserDefaults] 
objectForKey:@"EQTitles"]];
 
-    __weak typeof(self) _self = self;
+    __unsafe_unretained typeof(self) _self = self;
     [_popupPanel runModalForWindow:self.window completionHandler:^(NSInteger 
returnCode, NSInteger selectedIndex) {
 
         if (returnCode != NSOKButton)
diff --git a/modules/gui/macosx/VLCConvertAndSaveWindowController.m 
b/modules/gui/macosx/VLCConvertAndSaveWindowController.m
index 55000072b7..d03c1e679f 100644
--- a/modules/gui/macosx/VLCConvertAndSaveWindowController.m
+++ b/modules/gui/macosx/VLCConvertAndSaveWindowController.m
@@ -339,7 +339,7 @@
     [_popupPanel setCancelButtonString:_NS("Cancel")];
     [_popupPanel setPopupButtonContent:self.profileNames];
 
-    __weak typeof(self) _self = self;
+    __unsafe_unretained typeof(self) _self = self;
     [_popupPanel runModalForWindow:self.window completionHandler:^(NSInteger 
returnCode, NSInteger selectedIndex) {
 
         if (returnCode != NSOKButton)
@@ -475,7 +475,7 @@
     [_textfieldPanel setCancelButtonString: _NS("Cancel")];
     [_textfieldPanel setOkButtonString: _NS("Save")];
 
-    __weak typeof(self) _self = self;
+    __unsafe_unretained typeof(self) _self = self;
     [_textfieldPanel runModalForWindow:_customizePanel 
completionHandler:^(NSInteger returnCode, NSString *resultingText) {
         if (returnCode != NSOKButton || [resultingText length] == 0)
             return;

_______________________________________________
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits

Reply via email to