-
Notifications
You must be signed in to change notification settings - Fork 120
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
THREADINFOCLASS declares only 2 of many enum values #1439
Comments
The 2 values that are there appear to be the only 2 values defined in the SDK header. Are the others coming from the WDK? |
I don't know how to tell whether a given header file is in the SDK or WDK. But the link in the issue description shows just how many places this is defined, and nearly all of them have a much longer list than the metadata. The only 'short' def I could find didn't match the metadata either. |
The only headers that matter are the ones at https://github.com/microsoft/win32metadata/tree/main/generation/WinSDK/RecompiledIdlHeaders which we get from the public SDK NuGet packages. |
If that's the public SDK, how come this doc documents more than what's in the public SDK? |
Those are WDK docs. That may contain more definitions. |
When an enum has values that are split across two worlds like this, how do we see CsWin32 generating code when the WDK is available, when the 2 metadata winmd's each contain subsets of the same enum type? If one is 'complete' and one is partial, we still need a definitive plan for how to merge them in code generation. |
Unless the WDK is merged with the SDK metadata, overlapping types would presumably be moved entirely into the SDK and the WDK would then depend on the SDK e.g. |
Yes I'd expect any overlapping definitions would need to move to the SDK metadata. |
Keeping this issue here since shared definitions should be consolidated into the SDK metadata. |
Are these enums supposed to be int or uint? |
That's hard to say sometimes. I did find this in one of the Windows headers:
So maybe go with |
Thanks. |
The
THREADINFOCLASS
enum in the metadata defines only two of many enum values:metadata:
source
The text was updated successfully, but these errors were encountered: