Re: [PATCH libinput 1/3] Add msleep() helper function

2014-06-23 Thread Peter Hutterer
On Mon, Jun 23, 2014 at 01:06:05PM +0200, Hans de Goede wrote:
 Hi,
 
 The svg in the 3th patch contains a lot of [Not supported by viewer]
 text in the diamonds you use to mark if/else branches. It would be nice
 if you could fix this before pushing this set.

huh, weird. turns out draw.io has two of these diamonds, one in General, one
in Flowchart and only the latter seems to render correctly with text inside.
Fixed.
 
 Other then that this patch-set looks good and is:
 
 Reviewed-by: Hans de Goede hdego...@redhat.com

thanks.

Cheers,
   Peter
 
 On 06/23/2014 01:36 AM, Peter Hutterer wrote:
  For those whose eyes struggle to focus on 5 zeros in a row, or those just 
  sick
  of forgetting one zero and wondering why things don't work.
  
  Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
  ---
   src/libinput-util.h |  8 
   test/touchpad.c | 12 ++--
   2 files changed, 14 insertions(+), 6 deletions(-)
  
  diff --git a/src/libinput-util.h b/src/libinput-util.h
  index 11c4f5c..2558a3d 100644
  --- a/src/libinput-util.h
  +++ b/src/libinput-util.h
  @@ -23,6 +23,8 @@
   #ifndef LIBINPUT_UTIL_H
   #define LIBINPUT_UTIL_H
   
  +#include unistd.h
  +
   #include libinput.h
   
   void
  @@ -84,4 +86,10 @@ zalloc(size_t size)
  return calloc(1, size);
   }
   
  +static inline void
  +msleep(unsigned int ms)
  +{
  +   usleep(ms * 1000);
  +}
  +
   #endif /* LIBINPUT_UTIL_H */
  diff --git a/test/touchpad.c b/test/touchpad.c
  index 3565833..690e105 100644
  --- a/test/touchpad.c
  +++ b/test/touchpad.c
  @@ -125,7 +125,7 @@ START_TEST(touchpad_1fg_tap)
   
  assert_button_event(li, BTN_LEFT,
  LIBINPUT_BUTTON_STATE_PRESSED);
  -   usleep(30); /* tap-n-drag timeout */
  +   msleep(300); /* tap-n-drag timeout */
  assert_button_event(li, BTN_LEFT,
  LIBINPUT_BUTTON_STATE_RELEASED);
   
  @@ -177,7 +177,7 @@ START_TEST(touchpad_1fg_tap_n_drag)
   
  ck_assert_int_eq(libinput_next_event_type(li), LIBINPUT_EVENT_NONE);
   
  -   usleep(30); /* tap-n-drag timeout */
  +   msleep(300); /* tap-n-drag timeout */
   
  assert_button_event(li, BTN_LEFT,
  LIBINPUT_BUTTON_STATE_RELEASED);
  @@ -202,7 +202,7 @@ START_TEST(touchpad_2fg_tap)
   
  assert_button_event(li, BTN_RIGHT,
  LIBINPUT_BUTTON_STATE_PRESSED);
  -   usleep(30); /* tap-n-drag timeout */
  +   msleep(300); /* tap-n-drag timeout */
  assert_button_event(li, BTN_RIGHT,
  LIBINPUT_BUTTON_STATE_RELEASED);
   
  @@ -681,7 +681,7 @@ START_TEST(clickpad_softbutton_left_1st_fg_move)
  /* move out of the area, then wait for softbutton timer */
  litest_touch_move_to(dev, 0, 20, 90, 90, 20, 10);
  libinput_dispatch(li);
  -   usleep(40);
  +   msleep(400);
  libinput_dispatch(li);
  litest_drain_events(li);
   
  @@ -982,13 +982,13 @@ START_TEST(clickpad_topsoftbuttons_move_out_ignore)
   
  litest_touch_down(dev, 0, 50, 5);
  libinput_dispatch(li);
  -   usleep(20);
  +   msleep(200);
  libinput_dispatch(li);
  litest_assert_empty_queue(li);
   
  litest_touch_move_to(dev, 0, 50, 5, 80, 90, 20);
  libinput_dispatch(li);
  -   usleep(40);
  +   msleep(400);
  libinput_dispatch(li);
   
  litest_event(dev, EV_KEY, BTN_LEFT, 1);
  
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


[PATCH libinput 1/3] Add msleep() helper function

2014-06-22 Thread Peter Hutterer
For those whose eyes struggle to focus on 5 zeros in a row, or those just sick
of forgetting one zero and wondering why things don't work.

Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
 src/libinput-util.h |  8 
 test/touchpad.c | 12 ++--
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/src/libinput-util.h b/src/libinput-util.h
index 11c4f5c..2558a3d 100644
--- a/src/libinput-util.h
+++ b/src/libinput-util.h
@@ -23,6 +23,8 @@
 #ifndef LIBINPUT_UTIL_H
 #define LIBINPUT_UTIL_H
 
+#include unistd.h
+
 #include libinput.h
 
 void
@@ -84,4 +86,10 @@ zalloc(size_t size)
return calloc(1, size);
 }
 
+static inline void
+msleep(unsigned int ms)
+{
+   usleep(ms * 1000);
+}
+
 #endif /* LIBINPUT_UTIL_H */
diff --git a/test/touchpad.c b/test/touchpad.c
index 3565833..690e105 100644
--- a/test/touchpad.c
+++ b/test/touchpad.c
@@ -125,7 +125,7 @@ START_TEST(touchpad_1fg_tap)
 
assert_button_event(li, BTN_LEFT,
LIBINPUT_BUTTON_STATE_PRESSED);
-   usleep(30); /* tap-n-drag timeout */
+   msleep(300); /* tap-n-drag timeout */
assert_button_event(li, BTN_LEFT,
LIBINPUT_BUTTON_STATE_RELEASED);
 
@@ -177,7 +177,7 @@ START_TEST(touchpad_1fg_tap_n_drag)
 
ck_assert_int_eq(libinput_next_event_type(li), LIBINPUT_EVENT_NONE);
 
-   usleep(30); /* tap-n-drag timeout */
+   msleep(300); /* tap-n-drag timeout */
 
assert_button_event(li, BTN_LEFT,
LIBINPUT_BUTTON_STATE_RELEASED);
@@ -202,7 +202,7 @@ START_TEST(touchpad_2fg_tap)
 
assert_button_event(li, BTN_RIGHT,
LIBINPUT_BUTTON_STATE_PRESSED);
-   usleep(30); /* tap-n-drag timeout */
+   msleep(300); /* tap-n-drag timeout */
assert_button_event(li, BTN_RIGHT,
LIBINPUT_BUTTON_STATE_RELEASED);
 
@@ -681,7 +681,7 @@ START_TEST(clickpad_softbutton_left_1st_fg_move)
/* move out of the area, then wait for softbutton timer */
litest_touch_move_to(dev, 0, 20, 90, 90, 20, 10);
libinput_dispatch(li);
-   usleep(40);
+   msleep(400);
libinput_dispatch(li);
litest_drain_events(li);
 
@@ -982,13 +982,13 @@ START_TEST(clickpad_topsoftbuttons_move_out_ignore)
 
litest_touch_down(dev, 0, 50, 5);
libinput_dispatch(li);
-   usleep(20);
+   msleep(200);
libinput_dispatch(li);
litest_assert_empty_queue(li);
 
litest_touch_move_to(dev, 0, 50, 5, 80, 90, 20);
libinput_dispatch(li);
-   usleep(40);
+   msleep(400);
libinput_dispatch(li);
 
litest_event(dev, EV_KEY, BTN_LEFT, 1);
-- 
1.9.3

___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel