[PATCH spice-gtk v2] spice-widget: Use proper pointer type for channel_new/channel_destroy

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



No need to pass a generic pointer.
Avoid SpiceDisplay* -> gpointer* -> gpointer -> SpiceDisplay* cast
chain.

Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
---
 src/spice-widget.c | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

Changes since v1:
- merge 2 small patches changing pointer arguments.

diff --git a/src/spice-widget.c b/src/spice-widget.c
index ecd29e9d..97893591 100644
--- a/src/spice-widget.c
+++ b/src/spice-widget.c
@@ -111,8 +111,8 @@ static void update_mouse_grab(SpiceDisplay *display);
 static void try_mouse_grab(SpiceDisplay *display);
 static void try_mouse_ungrab(SpiceDisplay *display);
 static void recalc_geometry(GtkWidget *widget);
-static void channel_new(SpiceSession *s, SpiceChannel *channel, gpointer data);
-static void channel_destroy(SpiceSession *s, SpiceChannel *channel, gpointer data);
+static void channel_new(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *display);
+static void channel_destroy(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *display);
 static void cursor_invalidate(SpiceDisplay *display);
 static void update_area(SpiceDisplay *display, gint x, gint y, gint width, gint height);
 static void release_keys(SpiceDisplay *display);
@@ -713,13 +713,13 @@ spice_display_constructed(GObject *gobject)
     list = spice_session_get_channels(d->session);
     for (it = g_list_first(list); it != NULL; it = g_list_next(it)) {
         if (SPICE_IS_MAIN_CHANNEL(it->data)) {
-            channel_new(d->session, it->data, (gpointer*)display);
+            channel_new(d->session, it->data, display);
             break;
         }
     }
     for (it = g_list_first(list); it != NULL; it = g_list_next(it)) {
         if (!SPICE_IS_MAIN_CHANNEL(it->data))
-            channel_new(d->session, it->data, (gpointer*)display);
+            channel_new(d->session, it->data, display);
     }
     g_list_free(list);
 
@@ -2976,9 +2976,8 @@ static void gl_draw(SpiceDisplay *display,
 }
 #endif
 
-static void channel_new(SpiceSession *s, SpiceChannel *channel, gpointer data)
+static void channel_new(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *display)
 {
-    SpiceDisplay *display = data;
     SpiceDisplayPrivate *d = display->priv;
     int id;
 
@@ -3069,9 +3068,8 @@ static void channel_new(SpiceSession *s, SpiceChannel *channel, gpointer data)
     return;
 }
 
-static void channel_destroy(SpiceSession *s, SpiceChannel *channel, gpointer data)
+static void channel_destroy(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *display)
 {
-    SpiceDisplay *display = data;
     SpiceDisplayPrivate *d = display->priv;
     int id;
 
-- 
2.17.1

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]