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
Various sys_net improvements & bugfixes #9065
Conversation
fa51ba8
to
e34c742
Compare
// windows doesn't support getsockname for sockets that are not bound | ||
if (get_native_error() == 10022) | ||
{ | ||
memset(&native_addr, 0, native_addrlen); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Horrible hack, you can just save it in IDM instead of the operating system I think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree that windows is a horrible hack but I don't understand what you want to save in IDM?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sockname, dont rely on windows for it. unless it is expected to be empty for unbound sockets?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes it's expected to be 0.0.0.0:0 if unbound
e34c742
to
137fb56
Compare
Any thoughts on maybe consolidating the poll checks? |
No description provided.