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

Refactor part 5 #134

Merged
merged 17 commits into from
Nov 28, 2020
Merged

Refactor part 5 #134

merged 17 commits into from
Nov 28, 2020

Conversation

Thomasdezeeuw
Copy link
Collaborator

Refactors the recv and send and related functions, including the io::Read and io::Write implementations.

Also changes the Socket::new, Socket::pair and Socket::accept functions to set close-on-exec flag on Unix, on Apple platforms SOCK_NOSIGPIPE and on Windows the socket is made non-inheritable. New Socket::new_raw, Socket::pair_raw and Socket::accept_raw functions are added which do not set the above flags. This work as done by @de-vri-es in #117.

Thomasdezeeuw and others added 16 commits November 11, 2020 13:57
And recv_with_flags and recv_out_of_band.
Also add Socket::recv_vectored.
And add Socket::recv_from_vectored_with_flags.
This changes it to call Socket::recv. On unix this would previously call
read(2), but recv(2) without flags should have the same affect.

Furthermore this implements io::Read::read_vectored by calling
Socket::recv_vectored.
Socket::new, Socket::pair and Socket::accept now set close-on-exec flag
on Unix, on Apple platforms SOCK_NOSIGPIPE and on Windows the socket is
made non-inheritable.

New Socket::new_raw, Socket::pair_raw and Socket::accept_raw functions
are added which do not set the above flags.
@Thomasdezeeuw Thomasdezeeuw merged commit e431ce0 into rust-lang:master Nov 28, 2020
@Thomasdezeeuw Thomasdezeeuw deleted the refactor-p5 branch November 28, 2020 12:44
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.

2 participants