Re: winex11.drv: Strip the _XXX from X11DRV_GetKeyNameText when approperate

[Vitaliy Margolen]

On 01/17/2012 09:32 AM, Aric Stewart wrote:
> The problem is that there is no XK keycode for just Shift, Control or Menu 
> so X11DRV_MapVirtualKeyEx maps them to Shift_L, etc. As a result the string 
> XKeysymToString returns is a string with a _L or _R at the end, even if the 
> "don't care" bit is set.
> -
>            char *new_name;
> -
>            INT rc = 0;
> -
>            new_name = HeapAlloc(GetProcessHeap(),0,(idx-name)+1);
> -
>            lstrcpynA(new_name,name,(idx-name)+1);
> -
>            new_name[idx-name] = 0;
> -
>            TRACE("found scan=%04x keyc=%u keysym=%04x new_string=%s\n",
> -
>                   scanCode, keyc, (int)keys, new_name);
> -
>            if (lpBuffer && nSize)
> -
>                rc = MultiByteToWideChar(CP_UNIXCP, 0, new_name, -1, lpBuffer, nSize);
> -
>            HeapFree(GetProcessHeap(),0,new_name);
> -
>            return rc;

I don't think you need an extra buffer. Just do something like:
if (lpBuffer && nSize)
{
    rc = MultiByteToWideChar(CP_UNIXCP, 0, new_name, idx - name, lpBuffer, nSize);
    if (nSize > idx - name)
        lpBuffer[idx - name] = 0;
}

Of course need to account for rc.

Vitaliy