I had already sent a second patch without the C++-style comment. But regardless, if you say other parts of the code are also wrong, then I will leave this to someone closer to the project, who knows better what is it that should be done here.
2009/12/29 Dmitry Timoshkov <dmi...@codeweavers.com> > "David Quintana (gigaherz)" <gigah...@gmail.com> wrote: > > Issue was caused by VK_F4 having the same exact code as a lowercase 's'. >> So the code, which treated WM_CHAR and WM_KEYDOWN as the same, executed >> the wrong piece of code which was meant to be ONLY for WM_KEYDOWN. >> > > Thanks for spotting this bug. Anyway the fix is not correct. Virtual key > codes should be tested only in the WM_KEYDOWN handler, WM_CHAR handler > should test for character codes. Also, as Nikolay asked please don't use > C++ comments in Wine code. > > -- > Dmitry. >
