https://bugzilla.gnome.org/show_bug.cgi?id=768177

Jonas Ådahl <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #330600|none                        |accepted-commit_now
             status|                            |

--- Comment #4 from Jonas Ådahl <[email protected]> ---
Review of attachment 330600:

As far as I can tell, this looks correct to me (with one minor nit).

::: gdk/wayland/gdkselection-wayland.c
@@ +99,3 @@
 {
   /* Destination-side data */
+  SelectionData selections[3];

nit: Could just as well now add a ATOM_LAST or N_ATOMS to the enum, and use
that instead of the 3s? Makes it more obvious what is in the arrays as well.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
wayland-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-bugs

Reply via email to