<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 8, 2018 at 10:35 PM, Zebediah Figura <span dir="ltr"><<a href="mailto:z.figura12@gmail.com" target="_blank">z.figura12@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="m_8424102231328750670gmail-">On 04/03/18 23:04, Tom Watson wrote:<br></span><span class="m_8424102231328750670gmail-"><br>
</span>A few comments here:<br></blockquote><div><br></div><div>Thanks for your comments, being new here I cut and pasted an earlier example and used it as a base, I'll take them on board and see about resubmitting.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
...<br></blockquote><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+/*<br>
+ * Code temporarily disabled to pass the automated tests<br>
+ * as the code to hanlde PROGRESS_STOP/PROGRESS_CANCEL needs<br>
+ * to be installed on the vms<br>
+ */<br>
<br>
MoveFileWithProgress() is supported since Windows XP; I would be<br>
surprised if those flags are not implemented on native Windows.<br>
<br>
Rather, I suspect that your tests make assumptions about native<br>
implementation of CopyFile based on Wine's implementation. It's not<br>
necessarily true that Windows copies files in blocks of 65536 bytes. Did<br>
you check, when testing, that your callback ever actually ran?<br></blockquote><div> </div></div>I did test on testbot, at the time I did not realise they VMs there are actual systems I thought they were running the Wine dlls, you educated me to this earlier in the week, makes more sense now. The 65536 comes from Wine's kernel32/path.c,  <a href="https://source.winehq.org/git/wine.git/blob/HEAD:/dlls/kernel32/path.c#l1159" target="_blank">https://<wbr>source.winehq.org/git/wine.<wbr>git/blob/HEAD:/dlls/kernel32/<wbr>path.c#l1159</a> so I used it to test locally, they tests failed on testbot as it uses native win32. The test needs to be removed/rethought as none of the VMs actually call the progress function when copying a 128Mb file.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks again for your comments.</div><div class="gmail_extra"><br></div></div>