Re: [PATCH v2 0/2] MR145: kernel32: Unstub MoveFileWithProgress, add test.

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/kernelbase/file.c:
> }
>
> +/***********************************************************************
> -
> -
>       handle_progress_callback
> -
> - Wrapper for progress callback, if the return value is false, the copy or move operation should be cancelled
> - */
>
> +BOOL handle_progress_callback(LPPROGRESS_ROUTINE progress, unsigned stream_count, LARGE_INTEGER total, LARGE_INTEGER total_bytes_written, unsigned int buffer_size, LARGE_INTEGER stream_bytes_transferred, BOOL *invoke_progress,
> - DWORD callback_reason, HANDLE hSourceFile, HANDLE hDestinationFile, LPVOID param)

In general, please keep your code style consistent; you're mixing snake case with camel case. (We tend to prefer the former, and avoid Hungarian notation and LP* typedefs in new code.)