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