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

no POLLRDHUP in sys/poll.h for __APPLE__ #48

Closed
wants to merge 1 commit into from

Conversation

vsavkov
Copy link
Contributor

@vsavkov vsavkov commented Nov 16, 2024

No description provided.

@resetius
Copy link
Owner

@13hakta Could you remind me why you used POLLRDHUP here?

@resetius
Copy link
Owner

I 'fixed' it defining

 #if defined(__APPLE__) || defined(__FreeBSD__)
#define POLLRDHUP POLLHUP

@resetius resetius closed this Nov 16, 2024
@resetius
Copy link
Owner

72c1b1b

@13hakta
Copy link
Contributor

13hakta commented Nov 17, 2024

@13hakta Could you remind me why you used POLLRDHUP here?

It is used to handle remote disconnection socket event with poll.

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