https://bugzilla.gnome.org/show_bug.cgi?id=759168
Matthias Clasen <[email protected]> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #316924|none |reviewed
status| |
--- Comment #3 from Matthias Clasen <[email protected]> ---
Review of attachment 316924:
::: gdk/wayland/gdkdnd-wayland.c
@@ +51,3 @@
gdouble y;
+ gint prev_hot_x;
+ gint prev_hot_y;
I don't see the need to store these in the context. They can just be locals in
set_hot_spot
@@ +329,3 @@
+ .width = 1,
+ .height = 1,
+ },
This kind of on-the-fly struct looks a little out of style in the GTK+
codebase. Do we do this elsewhere in the Wayland backend ?
--
You are receiving this mail because:
You are on the CC list for the bug._______________________________________________
wayland-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-bugs
