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,

This function should be static.

This line is also very long, like some others; can you please split it a bit more?