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

sendto and sendTo are both public #100

Open
Joonsey opened this issue Feb 5, 2025 · 1 comment
Open

sendto and sendTo are both public #100

Joonsey opened this issue Feb 5, 2025 · 1 comment

Comments

@Joonsey
Copy link

Joonsey commented Feb 5, 2025

Socket.sendto should not be public?

I also think there is a typo in the docstring for Socket.sendto:

    // .darwin returns ISCONN error for already connected socket
    // .windows and .lynux both don't care
    // Intercepts ISCONN status for .darwin and retry sendto with null destination address
    pub fn sendto(
        /// The file descriptor of the sending socket.
        sockfd: std.posix.socket_t,
        /// Message to send.

.lynix should be .linux

@Joonsey
Copy link
Author

Joonsey commented Feb 5, 2025

I made a pull request: #101

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

No branches or pull requests

1 participant