Re: implement fixme in CreateAcceleratorTable

2004-02-27 Thread Pabs
On Sat, 2004-02-28 at 09:46, Dmitry Timoshkov wrote: > "Pabs" <[EMAIL PROTECTED]> wrote: > > > ChangeLog > > > > Added some checks to CreateAcceleratorTableA/W (removes fixmes) > > IMO it's better to use > > if (!MapVirtualKeyW(lpaccel[i].key, 0)) > { > WARN("Invalid VK\n"); > SetLast

Re: implement fixme in CreateAcceleratorTable

2004-02-27 Thread Dmitry Timoshkov
"Pabs" <[EMAIL PROTECTED]> wrote: > ChangeLog > > Added some checks to CreateAcceleratorTableA/W (removes fixmes) IMO it's better to use if (!MapVirtualKeyW(lpaccel[i].key, 0)) { WARN("Invalid VK\n"); SetLastError(ERROR_INVALID_PARAMETER); return NULL; } for a check whether VK is