[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 07/11] ui: rename cursor_{get->ref}, return it
From: |
marcandre . lureau |
Subject: |
[PATCH 07/11] ui: rename cursor_{get->ref}, return it |
Date: |
Mon, 30 Jan 2023 15:24:27 +0400 |
From: Marc-André Lureau <marcandre.lureau@redhat.com>
The naming is more conventional in QEMU code, and allows to simplify
some code.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
---
include/ui/console.h | 2 +-
ui/cursor.c | 3 ++-
ui/dbus-listener.c | 3 +--
ui/spice-display.c | 4 ++--
ui/vnc.c | 3 +--
5 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/include/ui/console.h b/include/ui/console.h
index ec28402a4f..3afb5c73a8 100644
--- a/include/ui/console.h
+++ b/include/ui/console.h
@@ -152,7 +152,7 @@ typedef struct QEMUCursor {
} QEMUCursor;
QEMUCursor *cursor_alloc(int width, int height);
-void cursor_get(QEMUCursor *c);
+QEMUCursor *cursor_ref(QEMUCursor *c);
void cursor_unref(QEMUCursor *c);
QEMUCursor *cursor_builtin_hidden(void);
QEMUCursor *cursor_builtin_left_ptr(void);
diff --git a/ui/cursor.c b/ui/cursor.c
index 31b09bf058..6fe67990e2 100644
--- a/ui/cursor.c
+++ b/ui/cursor.c
@@ -106,9 +106,10 @@ QEMUCursor *cursor_alloc(int width, int height)
return c;
}
-void cursor_get(QEMUCursor *c)
+QEMUCursor *cursor_ref(QEMUCursor *c)
{
c->refcount++;
+ return c;
}
void cursor_unref(QEMUCursor *c)
diff --git a/ui/dbus-listener.c b/ui/dbus-listener.c
index cc1054caff..f195d8df57 100644
--- a/ui/dbus-listener.c
+++ b/ui/dbus-listener.c
@@ -340,14 +340,13 @@ static void dbus_cursor_define(DisplayChangeListener *dcl,
DBusDisplayListener *ddl = container_of(dcl, DBusDisplayListener, dcl);
GVariant *v_data = NULL;
- cursor_get(c);
v_data = g_variant_new_from_data(
G_VARIANT_TYPE("ay"),
c->data,
c->width * c->height * 4,
TRUE,
(GDestroyNotify)cursor_unref,
- c);
+ cursor_ref(c));
qemu_dbus_display1_listener_call_cursor_define(
ddl->proxy,
diff --git a/ui/spice-display.c b/ui/spice-display.c
index 161868735b..918cadecf1 100644
--- a/ui/spice-display.c
+++ b/ui/spice-display.c
@@ -459,7 +459,7 @@ void qemu_spice_cursor_refresh_bh(void *opaque)
if (ssd->cursor) {
QEMUCursor *c = ssd->cursor;
assert(ssd->dcl.con);
- cursor_get(c);
+ cursor_ref(c);
qemu_mutex_unlock(&ssd->lock);
dpy_cursor_define(ssd->dcl.con, c);
qemu_mutex_lock(&ssd->lock);
@@ -764,7 +764,7 @@ static void display_mouse_define(DisplayChangeListener *dcl,
SimpleSpiceDisplay *ssd = container_of(dcl, SimpleSpiceDisplay, dcl);
qemu_mutex_lock(&ssd->lock);
- cursor_get(c);
+ cursor_ref(c);
cursor_unref(ssd->cursor);
ssd->cursor = c;
ssd->hot_x = c->hot_x;
diff --git a/ui/vnc.c b/ui/vnc.c
index 0bdcc3dfce..8aec5d751e 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -1032,8 +1032,7 @@ static void vnc_dpy_cursor_define(DisplayChangeListener
*dcl,
cursor_unref(vd->cursor);
g_free(vd->cursor_mask);
- vd->cursor = c;
- cursor_get(vd->cursor);
+ vd->cursor = cursor_ref(vd->cursor);
vd->cursor_msize = cursor_get_mono_bpl(c) * c->height;
vd->cursor_mask = g_malloc0(vd->cursor_msize);
cursor_get_mono_mask(c, 0, vd->cursor_mask);
--
2.39.1
- [PATCH 00/11] ui: dbus & misc fixes, marcandre . lureau, 2023/01/30
- [PATCH 01/11] ui/dbus: unregister clipboard on connection close, marcandre . lureau, 2023/01/30
- [PATCH 02/11] audio/dbus: there are no sender for p2p mode, marcandre . lureau, 2023/01/30
- [PATCH 03/11] ui/dbus: set mouse is-absolute during console creation, marcandre . lureau, 2023/01/30
- [PATCH 04/11] ui/dbus: update the display when switching surface, marcandre . lureau, 2023/01/30
- [PATCH 05/11] meson: ensure dbus-display generated code is built before other units, marcandre . lureau, 2023/01/30
- [PATCH 06/11] ui: rename cursor_{put->unref}, marcandre . lureau, 2023/01/30
- [PATCH 08/11] ui: keep current cursor with QemuConsole, marcandre . lureau, 2023/01/30
- [PATCH 10/11] ui: set cursor position upon listener registration, marcandre . lureau, 2023/01/30
- [PATCH 09/11] ui: set cursor upon listener registration, marcandre . lureau, 2023/01/30
- [PATCH 07/11] ui: rename cursor_{get->ref}, return it,
marcandre . lureau <=
- [PATCH 11/11] ui/sdl: get the GL context from the window, marcandre . lureau, 2023/01/30