GlobalMemoryStatusEx fails to meet API precondition

[sharwell]

Actual behavior

PInvoke.GlobalMemoryStatusEx is generated as follows:

/// <inheritdoc cref="GlobalMemoryStatusEx(winmdroot.System.SystemInformation.MEMORYSTATUSEX*)"/>
[SupportedOSPlatform("windows5.1.2600")]
internal static unsafe winmdroot.Foundation.BOOL GlobalMemoryStatusEx(out winmdroot.System.SystemInformation.MEMORYSTATUSEX lpBuffer)
{
    fixed (winmdroot.System.SystemInformation.MEMORYSTATUSEX* lpBufferLocal = &lpBuffer)
    {
        winmdroot.Foundation.BOOL __result = PInvoke.GlobalMemoryStatusEx(lpBufferLocal);
        return __result;
    }
}

Expected behavior

MEMORYSTATUSEX.dwLength is set prior to invoking the native method.

Repro steps

1. NativeMethods.txt content:

GlobalMemoryStatusEx

2. NativeMethods.json content (if present):

{
}

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

Context

• CsWin32 version: 0.3.49-beta
• Win32Metadata version (if explicitly set by project): Not set
• Target Framework: net6.0-windows
• LangVersion (if explicitly set by project): Not set

[AArnott]

Thanks for reporting this, @sharwell.
The metadata for this method is wrong. It attributes the parameter as [out] when it must be [in, out] to support the fact that it must be initialized by the caller first. In fact the docs for this method show it as in, out.

Once the metadata is fixed, CsWin32 will emit this as a ref parameter. It is up to the caller of the CsWin32 generated API to initialize the struct's length field, just as it would in C++.