Re: [PATCH 39/67] ui/console: fold text_console_update_cursor_timer

[Philippe Mathieu-Daudé]

On 30/8/23 11:38, marcandre.lureau@redhat.com wrote:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
>   ui/console.c | 10 ++--------
>   1 file changed, 2 insertions(+), 8 deletions(-)


>   static void text_console_update_cursor(void *opaque)
>   {
>       QemuConsole *s;
> @@ -2520,7 +2513,8 @@ static void text_console_update_cursor(void *opaque)
>       }
>   
>       if (count) {
> -        text_console_update_cursor_timer();
> +        timer_mod(cursor_timer, qemu_clock_get_ms(QEMU_CLOCK_REALTIME)
> +                  + CONSOLE_CURSOR_PERIOD / 2);

This should be padded here         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

>       }
>   }
>