[Wayland-bugs] [Bug 756729] No kinetic scrolling on Wayland

[gtk+ (GNOME Bugzilla)]

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

Carlos Garnacho <carlosg at gnome.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #313786|none                        |reviewed
             status|                            |

--- Comment #6 from Carlos Garnacho <carlosg at gnome.org> ---
Review of attachment 313786:

Looks quite alright to me, setting to "reviewed" mostly because this is on hold
till we get protocol bits merged. Only very minor formatting niggles (which I
know the wayland backend code is plagued of, but...).

::: gdk/wayland/gdkdevice-wayland.c
@@ +1080,2 @@
 static void
+flush_scroll_data (GdkWaylandDeviceData *device, GdkWaylandScrollData *scroll)

One argument per line, please :).

@@ +1173,3 @@
+
+static void
+pointer_handle_axis_source(void              *data,

Missing space before '(', variable names should also be aligned to the longest
arg.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-bugs/attachments/20151023/3af3ce12/attachment.html>