Really swap "x" and "y", not temporary copies.
Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
---
src/spice-session.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
I didn't test. The main question is why the current code is working?
Or is migration broken? Or we are not testing the type of migration
the original swapping code (manual) was supposed to make it work?
diff --git a/src/spice-session.c b/src/spice-session.c
index 04ba124a..d0d9e541 100644
--- a/src/spice-session.c
+++ b/src/spice-session.c
@@ -1742,12 +1742,9 @@ void spice_session_switching_disconnect(SpiceSession *self)
}
#define SWAP_STR(x, y) G_STMT_START { \
- const gchar *tmp; \
- const gchar *a = x; \
- const gchar *b = y; \
- tmp = a; \
- a = b; \
- b = tmp; \
+ gchar *tmp = x; \
+ x = y; \
+ y = tmp; \
} G_STMT_END
G_GNUC_INTERNAL
--
2.20.1
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel