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.

Sign up for GitHub

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

Jump to bottom

Pipe send should return whether the receiver might receive the packet #3128

Closed
eholk opened this issue Aug 7, 2012 · 1 comment
Closed

Pipe send should return whether the receiver might receive the packet #3128

eholk opened this issue Aug 7, 2012 · 1 comment
Labels
A-concurrency Area: Concurrency related issues.

Comments

@eholk
Copy link
Contributor

eholk commented Aug 7, 2012

Pipes can detect if the other end is closed. Receiving from a closed pipe reports this to the caller, but send does not. This is easy enough to add, and we should.
@ghost ghost assigned eholk Aug 7, 2012
@eholk
Copy link
Contributor Author

eholk commented Aug 7, 2012

So, send currently returns the next state of the protocol. I'm fixing this by instead generating try_ versions of all the send functions, which return an option(next_state), which is analogous to how the try_recv functions work.

@eholk eholk closed this as completed in abf4421 Aug 7, 2012
@eholk eholk removed their assignment Jun 16, 2014
flip1995 pushed a commit to flip1995/rust that referenced this issue Nov 3, 2020
@bors
Auto merge of rust-lang#6200 - rail-rain:borrowed_box_invalid_sugg, r…
74d8fbb 
…=phansch

fix the error-causing suggestion of 'borrowed_box'

Fixes rust-lang#3128

Fix the suggestion of 'borrowed_box', which causes a syntax error because it misses necessary parentheses.

---

changelog: Fix the error-causing suggestion of 'borrowed_box'
bors pushed a commit to rust-lang-ci/rust that referenced this issue May 15, 2021
@nrc
Don't align comments on extern crates
dd7add7 
Closes rust-lang#3128
bors pushed a commit to rust-lang-ci/rust that referenced this issue May 15, 2021
@nrc
test for rust-lang#3128
5dbe107
RalfJung pushed a commit to RalfJung/rust that referenced this issue Oct 21, 2023
@bors
Auto merge of rust-lang#3128 - rust-lang:dependabot/cargo/cargo-miri/…
fb39dd4 
…rustix-0.38.19, r=RalfJung

Bump rustix from 0.38.17 to 0.38.19 in /cargo-miri

Bumps [rustix](https://github.com/bytecodealliance/rustix) from 0.38.17 to 0.38.19.
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/bytecodealliance/rustix/commit/3a53dfe16cddc39fd20ecfb07c7d78880cb3880a"><code>3a53dfe</code></a> chore: Release rustix version 0.38.19</li>
<li><a href="https://github.com/bytecodealliance/rustix/commit/55cbe8839c8a78896b7f1e5310b3bcf20f8b0ac5"><code>55cbe88</code></a> Fixes for <code>Dir</code> on macOS, FreeBSD, and WASI.</li>
<li><a href="https://github.com/bytecodealliance/rustix/commit/31fd98ca723b93cc6101a3e29843ea5cf094e159"><code>31fd98c</code></a> Merge pull request from GHSA-c827-hfw6-qwvm</li>
<li><a href="https://github.com/bytecodealliance/rustix/commit/4bb2f4bbcd3a482112aadb4a226e0fce21fe5d8c"><code>4bb2f4b</code></a> Add <code>mlockall</code> and <code>munlockall</code> (<a href="https://redirect.github.com/bytecodealliance/rustix/issues/872">#872</a>)</li>
<li><a href="https://github.com/bytecodealliance/rustix/commit/d012bcf8967ba518bfa5a05d2c166139f4d4a9a0"><code>d012bcf</code></a> Fixes needed by io_uring. (<a href="https://redirect.github.com/bytecodealliance/rustix/issues/873">#873</a>)</li>
<li><a href="https://github.com/bytecodealliance/rustix/commit/5e5c0460cc576af9798f542a96ef6abb780b1ba1"><code>5e5c046</code></a> Add a test for <code>get_socket_nosigpipe</code>. (<a href="https://redirect.github.com/bytecodealliance/rustix/issues/874">#874</a>)</li>
<li><a href="https://github.com/bytecodealliance/rustix/commit/1a9d12963d49fda5347b622ee814e2e017fe7444"><code>1a9d129</code></a> chore: Release rustix version 0.38.18</li>
<li><a href="https://github.com/bytecodealliance/rustix/commit/112c268cd79438caae20a0bd4560ada5a1565450"><code>112c268</code></a> io-uring API updates. (<a href="https://redirect.github.com/bytecodealliance/rustix/issues/871">#871</a>)</li>
<li><a href="https://github.com/bytecodealliance/rustix/commit/0c227041a7faced80a532966dc085961fb46073a"><code>0c22704</code></a> Provide {get,set}_socket_nosigpipe on NetBSD and DragonFly BSD (<a href="https://redirect.github.com/bytecodealliance/rustix/issues/870">#870</a>)</li>
<li>See full diff in <a href="https://github.com/bytecodealliance/rustix/compare/v0.38.17...v0.38.19">compare view</a></li>
</ul>
</details>
<br />

[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=rustix&package-manager=cargo&previous-version=0.38.17&new-version=0.38.19)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting ``@dependabot` rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- ``@dependabot` rebase` will rebase this PR
- ``@dependabot` recreate` will recreate this PR, overwriting any edits that have been made to it
- ``@dependabot` merge` will merge this PR after your CI passes on it
- ``@dependabot` squash and merge` will squash and merge this PR after your CI passes on it
- ``@dependabot` cancel merge` will cancel a previously requested merge and block automerging
- ``@dependabot` reopen` will reopen this PR if it is closed
- ``@dependabot` close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- ``@dependabot` show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency
- ``@dependabot` ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- ``@dependabot` ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- ``@dependabot` ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
You can disable automated security fix PRs for this repo from the [Security Alerts page](https://github.com/rust-lang/miri/network/alerts).

</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-concurrency Area: Concurrency related issues.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eholk