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-26425] Always use thread-safe access to timers array #10360
Conversation
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.
@janvennemann Your logic looks good to me. You need to adjust this PR a bit e.g declaration and definition of variable timerLock should be out of TI_USE_KROLL_THREAD macro.
Can you create PR for 7_4_X so we can ask reporter to test? Thanks!
Addressed the missing declarations and moved them out of the kroll macro. |
@sgtcoolguy Updated the PR and the Jenkins build is looking good now. |
JIRA: https://jira.appcelerator.org/browse/TIMOB-26425
Optional Description:
Timers are always started in their own thread. Locking should also be done when running on main thread to make the access to the timers array thread-safe.