Re: [PATCH v3 0/7] MR4455: winewayland: Implement support for non-US keyboard layouts.

[Alexandros Frantzis (＠afrantzis)]

Alexandros Frantzis (@afrantzis) commented about dlls/winewayland.drv/wayland_keyboard.c:
> - if (!(layout = calloc(1, sizeof(*layout) + len)))
> - {
> -
>    ERR("Failed to allocate memory for Xkb layout entry\n");
> -
>    return;
> - }
> - ptr = (char *)(layout + 1);
> -
> - layout->xkb_layout = strcpy(ptr, xkb_layout);
> - ptr += len;
> -
> - layout->xkb_group = xkb_group;
> - layout->lang = lang;
> - layout->index = index;
> - if (index) layout->layout_id = next_layout_id++;
> -
> - TRACE("Created layout entry %p, hkl %04x%04x id %04x\n", layout, layout->index, layout->lang, layout->layout_id);

The use of "hkl" in this trace confused me, because I was expecting a proper `HKL` value. Perhaps we can find a more descriptive name, or even split this like `lang %04x index %04x`?