[PATCH 1/4] msvcp110: Add tr2_sys__File_size implementation and test.(try 3)

YongHao Hu christopherwuy at gmail.com
Thu Jun 11 10:48:30 CDT 2015


Thank you.
New patches had sent to wine-patches. : )

On 2015年06月11日 17:51, Piotr Caban wrote:
> On 06/11/15 11:31, YongHao Hu wrote:
>> Do you mean that  _Equivalent sets ERROR_SUCCESS when it succeeds?
> No. I mean that native doesn't really care about error returned by 
> GetLastError. It looks like it's just a side effect of the way how 
> it's implemented. Probably there's no need to be compatible here.
>
>> This one also needs to set the error explicitly, which I didn't do it.
>> Do you think that we should ignore GetLastError in these series of 
>> patches?
> I don't know, probably yes. The code is nicer when the error is not 
> set explicitly with SetLastError call. The functions from kernel32 
> will set the error anyway. It just will not match error set by native 
> msvcp120.
>
> Thanks,
> Piotr




More information about the wine-devel mailing list