<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi, Piotr.<br>
<br>
<div class="moz-cite-prefix">On 15/6/30 下午8:25, Piotr Caban wrote:<br>
</div>
<blockquote cite="mid:55928ACC.7030400@gmail.com" type="cite">
<blockquote type="cite">
<blockquote type="cite">Currently your implementation sometimes
returns error even so it
<br>
succeeds.
<br>
<br>
</blockquote>
Which one do you mean ? As you know, last_error hadling is not
correct
<br>
in wine's CopyFile, was this happened in Wine platform?
<br>
</blockquote>
{ "f1", "tr2_test_dir\\f1_copy", TRUE }
<br>
Your implementation returns error if f1_copy file already exists.
It only makes sense to use GetLastError if function have failed.
<br>
<br>
</blockquote>
I had tested the ret and the last_error before in testbot. The
result shows that the function's return value is exactly the same as
GetLastError().<br>
ret = p_tr2_sys__Copy_file(tests[i].source, tests[i].dest,
tests[i].fail_if_exists);<br>
printf("ret is %d\n", ret);<br>
printf("GetLastError is %d\n\n", GetLastError());<br>
<br>
Here is the test result:
<a class="moz-txt-link-freetext" href="https://testbot.winehq.org/JobDetails.pl?Key=14885&log_101=1#k101">https://testbot.winehq.org/JobDetails.pl?Key=14885&log_101=1#k101</a><br>
And { "f1", "tr2_test_dir\\f1_copy", TRUE } returns 80(
<meta charset="utf-8">
ERROR_FILE_EXSITS).<br>
<br>
</body>
</html>