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

std: Update documentation on the comm module #13465

Merged
merged 1 commit into from
Apr 17, 2014

Conversation

alexcrichton
Copy link
Member

Some of this documentation got a little out of date. There was no mention of a
SyncSender, and the entire "Outside the runtime" section isn't really true any
more (or really all that relevant).

This also updates a few other doc blocks and adds some examples.

//! respective operations without failing, however.
//!
//! ## Outside the Runtime
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this deletion because one can no longer use channels outside rust threads? I thought that you can, as long as you stay away from any method that blocks?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good point, I'll bring this back and clarify.

Some of this documentation got a little out of date. There was no mention of a
`SyncSender`, and the entire "Outside the runtime" section isn't really true any
more (or really all that relevant).

This also updates a few other doc blocks and adds some examples.
bors added a commit that referenced this pull request Apr 16, 2014
Some of this documentation got a little out of date. There was no mention of a
`SyncSender`, and the entire "Outside the runtime" section isn't really true any
more (or really all that relevant).

This also updates a few other doc blocks and adds some examples.
@bors bors closed this Apr 17, 2014
@bors bors merged commit d1bfa51 into rust-lang:master Apr 17, 2014
@alexcrichton alexcrichton deleted the fix-comm-dox branch April 18, 2014 22:12
bors added a commit that referenced this pull request Apr 19, 2014
…hton

I was getting a bit confused by these and (I think) managed to track it down to fallout from #13448 and #13465.
bors pushed a commit to rust-lang-ci/rust that referenced this pull request Jan 28, 2025
Closes rust-lang#13400.

changelog: [`unneeded_struct_pattern`]: Add new lint
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.

4 participants