Skip to content
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

Fix false positive in WorkQueue.addLast #2968

Merged
merged 1 commit into from
Nov 16, 2021

Conversation

dkhalanskyjb
Copy link
Collaborator

No description provided.

@qwwdfsad
Copy link
Contributor

qwwdfsad commented Oct 4, 2021

Any reason it is still draft?

@dkhalanskyjb
Copy link
Collaborator Author

I'm trying to reproduce it so that I have a test for this.

@dkhalanskyjb dkhalanskyjb marked this pull request as ready for review November 16, 2021 10:36
@dkhalanskyjb
Copy link
Collaborator Author

dkhalanskyjb commented Nov 16, 2021

Didn't find it in me after all to write a reproducer. Hopefully, the code is obviously correct, and in any case, it won't hurt anything.

@qwwdfsad qwwdfsad self-requested a review November 16, 2021 11:03
@dkhalanskyjb dkhalanskyjb merged commit e2a6827 into develop Nov 16, 2021
@dkhalanskyjb dkhalanskyjb deleted the fix-blockhound-false-positive branch November 16, 2021 13:23
yorickhenning pushed a commit to yorickhenning/kotlinx.coroutines that referenced this pull request Jan 28, 2022
pablobaxter pushed a commit to pablobaxter/kotlinx.coroutines that referenced this pull request Sep 14, 2022
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