-
Notifications
You must be signed in to change notification settings - Fork 625
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
NIOCore: replace mode_t
with CInt
#2132
Conversation
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.
As the CI failure indicates, mode_t
is not necessarily a CInt
. While we can change the way we hold the value internally, we can't change how we hold it in public API. Can we add a typealias that is defined differently (and appropriately) for Windows?
Sure a type alias would work. Any thoughts on a name? |
Windows does not have a `mode_t` type alias, instead using the de-sugared `CInt` type. De-sugar the instances to permit building on Windows.
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.
One nit and then we're good.
Co-authored-by: Cory Benfield <lukasa@apple.com>
Windows does not have a
mode_t
type alias, instead using thede-sugared
CInt
type. De-sugar the instances to permit building onWindows.