[PATCH 2/3]: msvcp110: Add tr2_sys__Rename implementation and test.(try 3)

YongHao Hu christopherwuy at gmail.com
Mon Jul 6 08:36:47 CDT 2015


On 15/7/6 下午4:46, Piotr Caban wrote:
> On 07/05/15 20:49, YongHao Hu wrote:
>> +    if(!strcmp(old_path, new_path) || MoveFileExA(old_path, 
> The comparison of paths doesn't look right. Why have you added it?
The test  { "tr2_test_dir\\f1", "tr2_test_dir\\f1", ERROR_SUCCESS } in 
wine's CopyFileA function set LastError as ERROR_ALREADY_EXISTS, so I 
just simply handled it by comparison of paths.
Your comment is true and the implementation was wrong. In fact, 
CopyFIleA doesn't change Last_error in this case.
Thus, we don't need the comparison of paths here. Ps: I think I will add 
to_do tests of these functions. : P
Thank you.

> Cheers,
> Piotr

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20150706/953c708f/attachment-0001.html>

More information about the wine-devel mailing list