-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Fix BlockHound false positives #2331
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
Conversation
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.
Please revisit a few more places on our code that rely on fast locking:
StateFlow
andSharedFlow
JobSupport
(finalizeFinishingState
& co.)
kotlinx-coroutines-debug/src/CoroutinesBlockHoundIntegration.kt
Outdated
Show resolved
Hide resolved
a0527e1
to
c18c9b1
Compare
Yeah, I completely missed that some code is protected using |
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.
Otherwise looks good, great job
No description provided.