-
Notifications
You must be signed in to change notification settings - Fork 10
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
NtCreateFile function enums #15
Comments
|
Keeping this issue here since shared definitions should be consolidated into the SDK metadata. |
I think we should remove this from the Win32 metadata and add it to the WDK metadata, since there are plenty other NtXxx APIs in the WDK but nothing comparable in the SDK. So the work here would be to remove it from this repo just as soon as the WDK repo add its, along with its friends. |
I will transfer to wdkmetadata then and add a task to the item to remove it from the SDK before completing it. |
So the function was removed from the win32 metadata and added to the wdk metadata? We'll need a matching release of both to avoid a missing function issue. |
The
NtCreateFile
method parameters could use some enum love:DesiredAccess
parameter is currently declared as uint, but it ought to be as thisFILE_ACCESS_FLAGS
enum.FileAttributes
parameter should be declared as a FILE_ATTRIBUTES enum. Previously suggested to beFILE_FLAGS_AND_ATTRIBUTES
but @KalleOlaviNiemitalo made a good comment in DesiredAccess and FILE_ACCESS_FLAGS enums win32metadata#1440 (comment) to consider.CreateOptions
(#1441)NtCreateFile
from the SDK metadata as part of this changeSee also discussion in microsoft/win32metadata#1440.
If
NtCreateFile
is in the WDK, it's already in the win32metadata, so... do we remove it, or fix the enums?The text was updated successfully, but these errors were encountered: