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

adding std.c.TCP.NODELAY for macos #22404

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mrjbq7
Copy link
Contributor

@mrjbq7 mrjbq7 commented Jan 4, 2025

This perhaps fixes #17260.

It adds constants for the TCP sockopt from darwin/bsd/netinet/tcp.h:

https://github.com/apple/darwin-xnu/blob/2ff845c2e033bd0ff64b5b6aa6063a1f8f65aa32/bsd/netinet/tcp.h#L208

@alexrp
Copy link
Member

alexrp commented Jan 4, 2025

The std.os changes are unnecessary; we don't support targeting macOS without libc because it doesn't have a stable syscall interface.

@mrjbq7 mrjbq7 changed the title adding std.posix.TCP.NODELAY for macos adding std.c.TCP.NODELAY for macos Jan 4, 2025
@mrjbq7
Copy link
Contributor Author

mrjbq7 commented Jan 4, 2025

Hi @alexrp i made the change and updated the title to show std.c.TCP for macos only.

@mrjbq7
Copy link
Contributor Author

mrjbq7 commented Jan 4, 2025

What are your thoughts on naming?

#define TCP_SENDMOREACKS 0x103

vs

pub const SEND_MORE_ACKS = 0x103

should we make it match better? SENDMOREACKS?

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.

error: root struct of file 'c' has no member named 'TCP'
2 participants