-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
iOS: Fix all memory leaks, Xcode-8 warnings & analyzer comments #8074
Conversation
… FIX-MEMORY_LEAKS # Conflicts: # iphone/Classes/SCListener.m # iphone/Classes/TiMediaAudioSession.m # iphone/Classes/TiMediaSoundProxy.m
… FIX-MEMORY_LEAKS # Conflicts: # iphone/Classes/MGSplitView/MGSplitViewController.m # iphone/iphone/Titanium.xcodeproj/project.pbxproj
return; | ||
} | ||
if (![ourContactStore executeSaveRequest:saveRequest error:&error]) { | ||
[self throwException:[NSString stringWithFormat:@"Unable to add member to group: %@",[TiUtils messageFromError:error]] | ||
subreason:nil | ||
location:CODELOCATION]; | ||
}; | ||
RELEASE_TO_NIL(saveRequest) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these changes to TiContactsGroup cannot be done. See
https://jira.appcelerator.org/browse/TIMOB-20010
It was done before and I had to revert it because this causes issues.
see this commit:
a0d4a50
If so, we would need to at least make a new PR for 92f08ca, maybe together with the updated default-entitlements that cause problems in Xcode 8. |
… FIX-MEMORY_LEAKS # Conflicts: # iphone/Classes/TiUITextField.m # iphone/Classes/TiUIiOSProxy.m
JIRA: https://jira.appcelerator.org/browse/TIMOB-23542