Re: [PATCH v2 0/5] MR5741: Draft: server: Implement key auto-repeat.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winewayland.drv/wayland_keyboard.c:
>                                      int rate, int delay)
>
> {
> -
>
> UINT speed;
> -
>
> TRACE("rate=%d delay=%d\n", rate, delay);
> -
>
> /* Handle non-negative rate values, ignore invalid (negative) values.  A
> -
> * rate of 0 disables repeat. */
> -
>
> if (rate >= 80) speed = 31;
> -
>
> else if (rate >= 5) speed = rate * 400 / 1000 - 1;
> -
>
> else speed = 0;
> -
> -
>
> delay = max(0, min(3, round(delay / 250.0) - 1));
> -
>
> NtUserSystemParametersInfo(SPI_SETKEYBOARDSPEED, speed, NULL, 0);
> -
>
> NtUserSystemParametersInfo(SPI_SETKEYBOARDDELAY, delay, NULL, 0);
> -
>
> NtUserCallOneParam(rate > 0, NtUserCallOneParam_SetKeyboardAutoRepeat);

For SPI integration, why not keep these calls and do the server request from win32u?