Skip to content

Channel: add ability to handle threads#30186

Merged
vtjnash merged 1 commit intomasterfrom
jn/channel-mt
Jan 7, 2019
Merged

Channel: add ability to handle threads#30186
vtjnash merged 1 commit intomasterfrom
jn/channel-mt

Conversation

@vtjnash
Copy link
Copy Markdown
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