Resend: shlwapi: Implemented StrToInt64ExA/W

carlo.bramix at libero.it carlo.bramix at libero.it
Thu Mar 8 07:35:56 CST 2012


According to the instructions that I just received at #winehq irc channel, I'm 
posting again the patch that I had sent before the release of version 1.4 since 
it had been freezed to "deferred" state.

Sincerely,

Carlo Bramini

>----Messaggio originale----
>Da: carlo.bramix at libero.it
>Data: 26/02/2012 23.19
>A: <wine-patches at winehq.org>
>Ogg: shlwapi: Implemented StrToInt64ExA/W
>
>New patch modified according to the newer suggestions that I received:
>1) The test on lpiRet could be removed for optimizing StrToIntExA() and 
>StrToIntExW(), but this will be logically different from the current 
>implementation in trunk, that's why it existed in previous patches. I did 
it, 
>but I do not know if there were some particular reason for having this...
>2) Simplified the StrToInt_results[] table and now the fields 
str_to_int64_ex 
>and str_to_int64_hex are the only ones, so StrToIntExA/W will share the 
same 
>entries of StrToInt64ExA/W, by casting them to 'int' type. Finally, 64 bit 
>constants have been generated with "shift by 32 with or" method in this 
table.
>3) Since it seems that StrToInt64ExA/W are not supported on Windows 2000, 
they 
>are loaded dynamically via GetProcAddress in the tests.
>
>Sincerely,
>
>Carlo Bramini.
>
>PS: Thank you very much to the friends that they are guiding me in this 
first 
>try.
>
>
>>I improved the patch according to the suggestions that I received:
>>1) Removed the C++ style comment
>>2) Removed the WARN message
>>3) Added the tests for the new functions. The test has been run 
successfully 
>>on Ubuntu 11.10.
>>
>>Sincerely
>>
>>Carlo Bramini
>>
>>>I tried to implement StrToInt64ExA/W as a reply to bug #27633
>>>I also corrected the comment in functions FormatInt() and FormatDouble() 
>>>because, according to MSDN, GetNumberFormat() returns the number of 
>>characters 
>>>written (or required) and not the number of bytes emitted:
>>>http://msdn.microsoft.com/en-us/library/windows/desktop/dd318110%28v=vs.
85%
>>29.
>>>aspx
>>>I hope my first patch to WINE is correct and it could be useful.
>>>Sincerely,
>>>
>>>Carlo Bramini.
>>>
>>
>>
>>
>
>

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: lst.txt
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20120308/6844d0f1/attachment.txt>


More information about the wine-patches mailing list