Re: [PATCH v3 0/1] MR6425: include: Name parameters in LPPROGRESS_ROUTINE typedef

[Alex Henrie (＠alexhenrie)]

Alex Henrie (@alexhenrie) commented about include/winbase.h:
> - This one seems to be a Win32 only definition. It also is defined with
> - WINAPI instead of CALLBACK in the windows headers.
>
> */
> -typedef DWORD (CALLBACK *LPPROGRESS_ROUTINE)(LARGE_INTEGER, LARGE_INTEGER, LARGE_INTEGER,
> -
>                                       LARGE_INTEGER, DWORD, DWORD, HANDLE,
> -
>                                       HANDLE, LPVOID);
>
> +typedef DWORD (CALLBACK *LPPROGRESS_ROUTINE)(
> - LARGE_INTEGER TotalFileSize,
> - LARGE_INTEGER TotalBytesTransferred,
> - LARGE_INTEGER StreamSize,
> - LARGE_INTEGER StreamBytesTransferred,
> - DWORD dwStreamNumber,
> - DWORD dwCallbackReason,
> - HANDLE hSourceFile,
> - HANDLE hDestinationFile,
> - LPVOID lpData

If we're going to do this, can we please use `void *` instead of `LPVOID`, name each parameter in snake_case, and not devote an entire line to each parameter?