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
Implement SDL-0187 Restructure iOS Threading / Queueing #1265
Implement SDL-0187 Restructure iOS Threading / Queueing #1265
Conversation
# Conflicts: # SmartDeviceLink/SDLStreamDelegate.m
# Conflicts: # SmartDeviceLink/SDLAudioStreamManager.m
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.
Left some comments
@@ -60,6 +60,11 @@ | |||
</BuildableReference> | |||
</MacroExpansion> | |||
<AdditionalOptions> | |||
<AdditionalOption | |||
key = "NSZombieEnabled" | |||
value = "YES" |
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.
This is for debugging, right?
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.
Yes, but since it's on the test scheme, it can just stay enabled.
@@ -92,7 +92,7 @@ - (void)sdl_sendFile:(SDLFile *)file mtuSize:(NSUInteger)mtuSize withCompletion: | |||
|
|||
// Wait for all packets be sent before returning whether or not the upload was a success | |||
__weak typeof(self) weakself = self; | |||
dispatch_group_notify(putFileGroup, dispatch_get_main_queue(), ^{ | |||
dispatch_group_notify(putFileGroup, [SDLGlobals sharedGlobals].sdlProcessingQueue, ^{ |
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.
Isn't this a breaking change, since devs will expect to get the response on the main thread?
* See todo for current location
Closed due to wrong branch name. New version is at #1348. |
Fixes #1028
This PR is ready for review.
Risk
This PR makes no API changes.
Testing Plan
Unit and smoke tests
Summary
This PR restructures the threading in the library to use an "execution context" structure.
Changelog
Enhancements
Tasks Remaining:
serialTransport
queue and use the processing queueconcurrentProcessing
queueCLA