[PATCH libinput 2/3] test: rename hover tests into semi_mt_hover
Peter Hutterer
peter.hutterer at who-t.net
Tue May 5 22:41:54 PDT 2015
On Thu, Apr 30, 2015 at 04:30:24PM -0400, Benjamin Tissoires wrote:
> From: Benjamin Tissoires <benjamin.tissoires at redhat.com>
>
> The current hover tests are uniquely designed for some Synaptics
> touchpad. Libinput can handle hovering through ABS_MT_DISTANCE,
> so we need to reserve the "hover" name for real hovering devices.
>
> Signed-off-by: Benjamin Tissoires <benjamin.tissoires at redhat.com>
merged and pushed 1 and 2 of this series, thanks.
Cheers,
Peter
> ---
> test/touchpad.c | 26 +++++++++++++-------------
> 1 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/test/touchpad.c b/test/touchpad.c
> index 830e589..49f5023 100644
> --- a/test/touchpad.c
> +++ b/test/touchpad.c
> @@ -3480,7 +3480,7 @@ hover_start(struct litest_device *dev, unsigned int slot,
> /* WARNING: no SYN_REPORT! */
> }
>
> -START_TEST(touchpad_hover_noevent)
> +START_TEST(touchpad_semi_mt_hover_noevent)
> {
> struct litest_device *dev = litest_current_device();
> struct libinput *li = dev->libinput;
> @@ -3511,7 +3511,7 @@ START_TEST(touchpad_hover_noevent)
> }
> END_TEST
>
> -START_TEST(touchpad_hover_down)
> +START_TEST(touchpad_semi_mt_hover_down)
> {
> struct litest_device *dev = litest_current_device();
> struct libinput *li = dev->libinput;
> @@ -3586,7 +3586,7 @@ START_TEST(touchpad_hover_down)
> }
> END_TEST
>
> -START_TEST(touchpad_hover_down_hover_down)
> +START_TEST(touchpad_semi_mt_hover_down_hover_down)
> {
> struct litest_device *dev = litest_current_device();
> struct libinput *li = dev->libinput;
> @@ -3673,7 +3673,7 @@ START_TEST(touchpad_hover_down_hover_down)
> }
> END_TEST
>
> -START_TEST(touchpad_hover_down_up)
> +START_TEST(touchpad_semi_mt_hover_down_up)
> {
> struct litest_device *dev = litest_current_device();
> struct libinput *li = dev->libinput;
> @@ -3740,7 +3740,7 @@ START_TEST(touchpad_hover_down_up)
> }
> END_TEST
>
> -START_TEST(touchpad_hover_2fg_noevent)
> +START_TEST(touchpad_semi_mt_hover_2fg_noevent)
> {
> struct litest_device *dev = litest_current_device();
> struct libinput *li = dev->libinput;
> @@ -3785,7 +3785,7 @@ START_TEST(touchpad_hover_2fg_noevent)
> }
> END_TEST
>
> -START_TEST(touchpad_hover_2fg_1fg_down)
> +START_TEST(touchpad_semi_mt_hover_2fg_1fg_down)
> {
> struct litest_device *dev = litest_current_device();
> struct libinput *li = dev->libinput;
> @@ -4232,14 +4232,14 @@ int main(int argc, char **argv) {
> litest_add("touchpad:left-handed", touchpad_left_handed_delayed, LITEST_TOUCHPAD|LITEST_BUTTON, LITEST_CLICKPAD);
> litest_add("touchpad:left-handed", touchpad_left_handed_clickpad_delayed, LITEST_CLICKPAD, LITEST_APPLE_CLICKPAD);
>
> - /* Hover tests aren't generic, they only work on this device and
> + /* Semi-MT hover tests aren't generic, they only work on this device and
> * ignore the semi-mt capability (it doesn't matter for the tests */
> - litest_add_for_device("touchpad:hover", touchpad_hover_noevent, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> - litest_add_for_device("touchpad:hover", touchpad_hover_down, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> - litest_add_for_device("touchpad:hover", touchpad_hover_down_up, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> - litest_add_for_device("touchpad:hover", touchpad_hover_down_hover_down, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> - litest_add_for_device("touchpad:hover", touchpad_hover_2fg_noevent, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> - litest_add_for_device("touchpad:hover", touchpad_hover_2fg_1fg_down, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> + litest_add_for_device("touchpad:semi-mt-hover", touchpad_semi_mt_hover_noevent, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> + litest_add_for_device("touchpad:semi-mt-hover", touchpad_semi_mt_hover_down, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> + litest_add_for_device("touchpad:semi-mt-hover", touchpad_semi_mt_hover_down_up, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> + litest_add_for_device("touchpad:semi-mt-hover", touchpad_semi_mt_hover_down_hover_down, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> + litest_add_for_device("touchpad:semi-mt-hover", touchpad_semi_mt_hover_2fg_noevent, LITEST_SYNAPTICS_HOVER_SEMI_MT);
> + litest_add_for_device("touchpad:semi-mt-hover", touchpad_semi_mt_hover_2fg_1fg_down, LITEST_SYNAPTICS_HOVER_SEMI_MT);
>
> litest_add_for_device("touchpad:trackpoint", touchpad_trackpoint_buttons, LITEST_SYNAPTICS_TRACKPOINT_BUTTONS);
> litest_add_for_device("touchpad:trackpoint", touchpad_trackpoint_mb_scroll, LITEST_SYNAPTICS_TRACKPOINT_BUTTONS);
> --
> 1.7.1
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
More information about the wayland-devel
mailing list