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
[TIMOB-25836] : iOS Using "run-on-main-thread", multiple global event listeners do not work #9908
Conversation
… listeners do not work
Please add unit tests for this. I think the example from JIRA will geko already, together with a finish() |
@@ -188,7 +188,7 @@ - (void)fireEvent:(NSArray *)args | |||
|
|||
DebugLog(@"[DEBUG] Firing app event: %@", type); | |||
|
|||
NSArray *array = [appListeners objectForKey:type]; | |||
NSArray *array = [[appListeners objectForKey:type] copy]; |
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.
I wonder if we should rather synchronize the appListeners
setters, so nothing can be mutated during its write-process.
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.
I think it will slow the processing. And if there are not more use cases like this, we should avoid.
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.
Ok then, taking your word!
Generated by 🚫 dangerJS |
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.
CR approved.
https://jira.appcelerator.org/browse/TIMOB-25836