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

[Philippe Mathieu-Daudé]

On 30/8/23 11:38, marcandre.lur...@redhat.com wrote:

From: Marc-André Lureau <marcandre.lur...@redhat.com>

Signed-off-by: Marc-André Lureau <marcandre.lur...@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é <phi...@linaro.org>

      }
  }