Skip to content

Conversation

@popoaichuiniu
Copy link
Collaborator

@popoaichuiniu popoaichuiniu commented Apr 16, 2025

A UsbClient must call stop() before destruction to prevent threads from accessing its resources after it's destroyed.

issue: #42

…uring WorkThreadExecutor shutdown

	A UsbClient must call stop() before destruction to prevent threads from accessing its resources after it's destroyed.

issue: lynx-family#42
@popoaichuiniu popoaichuiniu force-pushed the p/zhoumingsong.smile/fix_worker_join_crash branch from 4290c15 to 1202824 Compare April 16, 2025 12:37
@popoaichuiniu popoaichuiniu changed the title [BugFix][android][ios][common] Fix crash when joining worker thread d… [BugFix][android][ios][common] Fix crash when joining worker thread during WorkThreadExecutor shutdown Apr 16, 2025
@popoaichuiniu popoaichuiniu changed the title [BugFix][android][ios][common] Fix crash when joining worker thread during WorkThreadExecutor shutdown [BugFix][android][ios][common] Fix crash when exec worker.join during WorkThreadExecutor shutdown Apr 16, 2025
@popoaichuiniu popoaichuiniu changed the title [BugFix][android][ios][common] Fix crash when exec worker.join during WorkThreadExecutor shutdown [BugFix][android][ios][common] Fix crash when exec worker.join() during WorkThreadExecutor shutdown Apr 16, 2025
@popoaichuiniu popoaichuiniu changed the title [BugFix][android][ios][common] Fix crash when exec worker.join() during WorkThreadExecutor shutdown [BugFix][android][ios][common] Fix crash when executing worker.join() during WorkThreadExecutor shutdown. Apr 16, 2025
@popoaichuiniu popoaichuiniu changed the title [BugFix][android][ios][common] Fix crash when executing worker.join() during WorkThreadExecutor shutdown. [BugFix][android][ios][common] Fix thread crash that occurs when calling worker.join() during WorkThreadExecutor shutdown. Apr 16, 2025
@RovicnowYoyi RovicnowYoyi merged commit d2479ed into lynx-family:main Apr 17, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants