Use fabs() instead of abs() on double values.

Silences clang warnings.

Signed-off-by: Matthieu Herrb <matthieu@herrb.eu>
Reviewed-by: Walter Harms <wharms@bfs.de>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
This commit is contained in:
Matthieu Herrb
2019-02-10 17:29:22 +01:00
committed by Peter Hutterer
parent f8d896ddca
commit a4e5f74b58

View File

@@ -2599,7 +2599,7 @@ HandleScrolling(SynapticsPrivate * priv, struct SynapticsHwState *hw,
priv->scroll.delta_y += priv->scroll.coast_speed_y * dtime * abs(para->scroll_dist_vert);
delay = MIN(delay, POLL_MS);
if (abs(priv->scroll.coast_speed_y) < ddy) {
if (fabs(priv->scroll.coast_speed_y) < ddy) {
priv->scroll.coast_speed_y = 0;
priv->scroll.packets_this_scroll = 0;
}
@@ -2614,7 +2614,7 @@ HandleScrolling(SynapticsPrivate * priv, struct SynapticsHwState *hw,
double ddx = para->coasting_friction * dtime;
priv->scroll.delta_x += priv->scroll.coast_speed_x * dtime * abs(para->scroll_dist_horiz);
delay = MIN(delay, POLL_MS);
if (abs(priv->scroll.coast_speed_x) < ddx) {
if (fabs(priv->scroll.coast_speed_x) < ddx) {
priv->scroll.coast_speed_x = 0;
priv->scroll.packets_this_scroll = 0;
}
@@ -2670,8 +2670,8 @@ clickpad_guess_clickfingers(SynapticsPrivate * priv,
* really, this should be dependent on the touchpad size. Also,
* you'll need to find a touchpad that doesn't lie about it's
* size. Good luck. */
if (abs(x1 - x2) < (priv->maxx - priv->minx) * .3 &&
abs(y1 - y2) < (priv->maxy - priv->miny) * .3) {
if (fabs(x1 - x2) < (priv->maxx - priv->minx) * .3 &&
fabs(y1 - y2) < (priv->maxy - priv->miny) * .3) {
close_point |= (1 << j);
close_point |= (1 << i);
}