[PATCH libinput 1/3] test: move enable_edge_scroll up

Hans de Goede hdegoede at redhat.com
Wed Jul 8 05:58:19 PDT 2015


Hi,

On 06-07-15 06:26, Peter Hutterer wrote:
> No functional changes, just so we can group those helpers together.
>
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>

The entire series LGTM:

Reviewed-by: Hans de Goede <hdegoede at redhat.com>

Regards,

Hans


> ---
>   test/touchpad.c | 26 +++++++++++++-------------
>   1 file changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/test/touchpad.c b/test/touchpad.c
> index 1f11cfa..082312c 100644
> --- a/test/touchpad.c
> +++ b/test/touchpad.c
> @@ -32,6 +32,19 @@
>   #include "libinput-util.h"
>   #include "litest.h"
>
> +static void
> +enable_edge_scroll(struct litest_device *dev)
> +{
> +	enum libinput_config_status status, expected;
> +	struct libinput_device *device = dev->libinput_device;
> +
> +	status = libinput_device_config_scroll_set_method(device,
> +					  LIBINPUT_CONFIG_SCROLL_EDGE);
> +
> +	expected = LIBINPUT_CONFIG_STATUS_SUCCESS;
> +	litest_assert_int_eq(status, expected);
> +}
> +
>   START_TEST(touchpad_1fg_motion)
>   {
>   	struct litest_device *dev = litest_current_device();
> @@ -1442,19 +1455,6 @@ START_TEST(touchpad_scroll_natural)
>   }
>   END_TEST
>
> -static void
> -enable_edge_scroll(struct litest_device *dev)
> -{
> -	enum libinput_config_status status, expected;
> -	struct libinput_device *device = dev->libinput_device;
> -
> -	status = libinput_device_config_scroll_set_method(device,
> -					  LIBINPUT_CONFIG_SCROLL_EDGE);
> -
> -	expected = LIBINPUT_CONFIG_STATUS_SUCCESS;
> -	litest_assert_int_eq(status, expected);
> -}
> -
>   START_TEST(touchpad_edge_scroll)
>   {
>   	struct litest_device *dev = litest_current_device();
>


More information about the wayland-devel mailing list