On 9/30/2013 00:51, Daniel Jeliński wrote:
+struct progress_list { + const DWORD progress_retval_init; /* value to return from progress routine */ + const BOOL cancel_init; /* value to set Cancel flag to */ + const DWORD progress_retval_end; /* value to return from progress routine */ + const BOOL cancel_end; /* value to set Cancel flag to */ + const DWORD progress_count; /* number of times progress is invoked */ + const BOOL copy_retval; /* expected CopyFileEx result */ + const DWORD lastError; /* expected CopyFileEx error code */ +} ; I don't see a point making them 'const'.
+static DWORD WINAPI progress(LARGE_INTEGER TotalFileSize, + LARGE_INTEGER TotalBytesTransferred, + LARGE_INTEGER StreamSize, + LARGE_INTEGER StreamBytesTransferred, + DWORD dwStreamNumber, + DWORD dwCallbackReason, + HANDLE hSourceFile, + HANDLE hDestinationFile, + LPVOID lpData) +{ + progressInvoked++; Please pass all globals as context data with lpData, and please use 'void*' instead of LPVOID.