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

Update documentation for close, send and recv #186

Closed
wants to merge 1 commit into from

Conversation

Yuras
Copy link
Contributor

@Yuras Yuras commented Feb 7, 2016

Add a note that sending data to or receiving data from closed
socket in unsafe. See #169

I'd prefer to fix the implementation instead, but it is not trivial and requires deeper knowledges of the code base. Fixing the documentation is definitely better then doing nothing.

@eborden eborden mentioned this pull request Jun 6, 2016
@eborden
Copy link
Collaborator

eborden commented Jun 12, 2016

The path agreed upon in #169 makes this documentation update relevant. The documentation in Network.Socket.ByteString and Network.Socket.ByteString.Lazy should also be updated for this to be merged. @Yuras, let me know if you are interested in fulfilling this.

Add a note that sending data to or receiving data from closed
socket in unsafe. See haskell#169
@Yuras
Copy link
Contributor Author

Yuras commented Jun 15, 2016

@eborden I updated the PR, please review

@kazu-yamamoto
Copy link
Collaborator

I close this since this pull request conflicts. But I will update the documents as @Yuras wishes.
I know that #211 also include the similar commit. But I will do by myself with their credit preserved.

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.

3 participants