Add a normalized_length helper function and use this where applicable, just a minor cleanup.
Signed-off-by: Hans de Goede <hdego...@redhat.com> --- src/evdev-mt-touchpad-tap.c | 10 +++------- src/filter.c | 4 +--- src/libinput-private.h | 7 +++++++ 3 files changed, 11 insertions(+), 10 deletions(-) diff --git a/src/evdev-mt-touchpad-tap.c b/src/evdev-mt-touchpad-tap.c index cb8fa2e..6a1ada5 100644 --- a/src/evdev-mt-touchpad-tap.c +++ b/src/evdev-mt-touchpad-tap.c @@ -530,14 +530,10 @@ static bool tp_tap_exceeds_motion_threshold(struct tp_dispatch *tp, struct tp_touch *t) { - int threshold = DEFAULT_TAP_MOVE_THRESHOLD; - struct normalized_coords normalized; + struct normalized_coords norm = + tp_normalize_delta(tp, device_delta(t->point, t->tap.initial)); - normalized = tp_normalize_delta(tp, - device_delta(t->point, t->tap.initial)); - - return normalized.x * normalized.x + normalized.y * normalized.y - > threshold * threshold; + return normalized_length(norm) > DEFAULT_TAP_MOVE_THRESHOLD; } static bool diff --git a/src/filter.c b/src/filter.c index dc29928..dd4bd58 100644 --- a/src/filter.c +++ b/src/filter.c @@ -137,11 +137,9 @@ tracker_by_offset(struct pointer_accelerator *accel, unsigned int offset) static double calculate_tracker_velocity(struct pointer_tracker *tracker, uint64_t time) { - double distance; double tdelta = time - tracker->time + 1; - distance = hypot(tracker->delta.x, tracker->delta.y); - return distance / tdelta; /* units/ms */ + return normalized_length(tracker->delta) / tdelta; /* units/ms */ } static double diff --git a/src/libinput-private.h b/src/libinput-private.h index ec5caf6..4b3c174 100644 --- a/src/libinput-private.h +++ b/src/libinput-private.h @@ -24,6 +24,7 @@ #define LIBINPUT_PRIVATE_H #include <errno.h> +#include <math.h> #include "linux/input.h" @@ -373,4 +374,10 @@ device_delta(struct device_coords a, struct device_coords b) return delta; } +static inline double +normalized_length(struct normalized_coords norm) +{ + return hypot(norm.x, norm.y); +} + #endif /* LIBINPUT_PRIVATE_H */ -- 2.3.3 _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel