Regression for `WriteFile` friendly overload

[alexrp]

Actual behavior

Expected behavior

This overload used to take a byte*. byte? is definitely not right.

Repro steps

1. NativeMethods.txt content:

   WriteFile

2. NativeMethods.json content (if present):

   {
   "className": "WindowsPInvoke",
   "allowMarshaling": false,
   "wideCharOnly": false,
   "emitSingleFile": true
   }

Context

• CsWin32 version: 0.2.206-beta
• Win32Metadata version (if explicitly set by project): 48.0.19-preview
• Target Framework: net7.0
• LangVersion (if explicitly set by project): latest

[AArnott]

Thank you for the report. I see in the metadata that this API is declared as:

public unsafe static extern BOOL WriteFile([In] HANDLE hFile, [Optional][In][Const][MemorySize(BytesParamIndex = 2)] byte* lpBuffer, [In] uint nNumberOfBytesToWrite, [Optional][Out] uint* lpNumberOfBytesWritten, [Optional][In][Out] OVERLAPPED* lpOverlapped);

My guess is because lpBuffer isn't using the NativeArrayInfoAttribute that we decide it isn't an array. I'm not sure if the metadata should be using one or the other, but CsWin32 can be updated to recognize MemorySize as well.