[prev in list] [next in list] [prev in thread] [next in thread]
List: wine-devel
Subject: Re: riched20: explicitly cast WPARAM to character types
From: Thomas Faber <thfabba () gmx ! de>
Date: 2011-10-24 18:10:50
Message-ID: 4EA5AA2A.1060901 () gmx ! de
[Download RAW message or body]
Hi Michael,
On 2011-10-23 21:25, Michael Stefaniuc wrote:
> On 10/22/2011 10:38 PM, Thomas Faber wrote:
>> This makes the use of casts more consistent and shows the conversion
>> from WPARAM to CHAR/WCHAR is intentional for WM_CHAR/EM_SETPASSWORDCHAR.
>> Also fixes two MSVC warnings.
> To make it consistent remove the WCHAR cast 3 lines above.
> Casts are *not* there to show a conversion is intentional. They tell the
> compiler to STFU and to the human reader they mean "*WARNING* doing
> something odd here". So don't cry wolf when there's no wolf in sight.
>
> No clue anyway why MSVC even warns, integer to integer conversions are
> allowed. If you still want to get rid of those warnings the fix would be
> to change the type of charCode to WCHAR. To correctly unmarshal the
> WCHAR out of the WPARAM when passing it to ME_Char() use LOWORD().
good point, that sounds like a more appropriate solution. I'll see to it
when I find the time.
Thanks.
-Tom
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic