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.

Sign up for GitHub

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

Jump to bottom

MF_VERSION missing from metadata #1431

Closed
diontools opened this issue Jan 15, 2023 · 2 comments
Closed

MF_VERSION missing from metadata #1431

diontools opened this issue Jan 15, 2023 · 2 comments
Assignees
@mikebattista
Labels
bug Something isn't working missing api Some documented API is missing from the metadata

Comments

@diontools
Copy link

Actual behavior

CsWin32 cannot generate the const MF_VERSION.

Expected behavior

MF_VERSION is declared in PInvoke.

Repro steps

  1. NativeMethods.txt content:
MF_VERSION

  1. NativeMethods.json content (if present):

  2. Any of your own code that should be shared?

Context

  • CsWin32 version: 0.2.164-beta
  • Win32Metadata version (if explicitly set by project):
  • Target Framework: net7.0-windows10.0.19041.0
  • LangVersion (if explicitly set by project):
@diontools diontools added the bug Something isn't working label Jan 15, 2023
@AArnott AArnott transferred this issue from microsoft/CsWin32 Jan 17, 2023
@AArnott AArnott changed the title Cannot generate MF_VERSION. MF_VERSION missing from metadata Jan 17, 2023
@mikebattista mikebattista self-assigned this Jan 17, 2023
@mikebattista mikebattista added the missing api Some documented API is missing from the metadata label Jan 17, 2023
@mikebattista
Copy link
Collaborator

mikebattista commented Jan 20, 2023 •
edited
Loading

Is public const uint MF_VERSION = 131184u; the correct value?

I defined it as public const uint MF_VERSION = (MF_SDK_VERSION << 16 | MF_API_VERSION); and the above is what's showing in the metadata.

@riverar
Copy link
Collaborator

riverar commented Jan 21, 2023

@mikebattista Can confirm. Holds true for 22621 SDK too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mikebattista mikebattista

Labels
bug Something isn't working missing api Some documented API is missing from the metadata
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@riverar @diontools @mikebattista