Skip to content

Conversation

@antrik
Copy link
Contributor

@antrik antrik commented Nov 4, 2017

Two fixes (along with test cases) for handling of sender/receiver closed conditions.

"Broken Pipe" means the receiver end is closed -- it's just wrong to
signal this error when receiving from a closed sender.

(Admittedly, `Connection Reset` for closed sender is rather arbitrary: a
closed sender is normally signalled as EOF, not as an error, by the
OS... But it's what the other back-ends use -- and it's better than
"Broken Pipe" at any rate.)
@jdm
Copy link
Member

jdm commented Nov 4, 2017

@bors-servo r+
Good find!

@bors-servo
Copy link
Contributor

📌 Commit e438c8a has been approved by jdm

@bors-servo
Copy link
Contributor

⌛ Testing commit e438c8a with merge c3fb5d7...

bors-servo pushed a commit that referenced this pull request Nov 4, 2017
Fix error handling in `inprocess` back-end

Two fixes (along with test cases) for handling of sender/receiver closed conditions.
@bors-servo
Copy link
Contributor

☀️ Test successful - status-appveyor, status-travis
Approved by: jdm
Pushing c3fb5d7 to master...

@bors-servo bors-servo merged commit e438c8a into servo:master Nov 4, 2017
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.

5 participants