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

Channel: add ability to handle threads #30186

Merged
merged 1 commit into from
Jan 7, 2019
Merged

Channel: add ability to handle threads #30186

merged 1 commit into from
Jan 7, 2019

Conversation

vtjnash
Copy link
Member

@vtjnash vtjnash commented Nov 28, 2018

Continuing from #30061, this internally changes Channel to be thread-safe (keeping the same, or slightly stronger tests). Adopted from #22631.

@vtjnash vtjnash added the multithreading Base.Threads and related functionality label Nov 28, 2018
@vtjnash vtjnash force-pushed the jn/condition-mt branch 4 times, most recently from 06caab5 to fef54a7 Compare December 11, 2018 19:12
@vtjnash vtjnash force-pushed the jn/condition-mt branch 4 times, most recently from b0c2e18 to a6ea3ef Compare December 17, 2018 02:48
@vtjnash vtjnash changed the base branch from jn/condition-mt to master December 17, 2018 17:08
This drops the internal `notify_error` API, use `close` instead.
@vtjnash vtjnash merged commit 9619079 into master Jan 7, 2019
@vtjnash vtjnash deleted the jn/channel-mt branch January 7, 2019 21:15
Keno pushed a commit that referenced this pull request Jun 5, 2024
This drops the internal `notify_error` API, use `close` instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
multithreading Base.Threads and related functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants