From: Peter Hutterer Date: Tue, 8 Jul 2014 02:05:36 +0000 (+1000) Subject: filter: use a separate variable for the final accel factor X-Git-Tag: 0.5.0~35 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b908e070e9a81e5ab6fa7840ce085a330a2c7eac;p=platform%2Fupstream%2Flibinput.git filter: use a separate variable for the final accel factor velocity is in unit/ms, the threshold is in units/ms. Once we divide velocity/threshold, we're not in units/ms anymore but have a unitless factor. Use a separate variable to avoid confusion. Signed-off-by: Peter Hutterer Reviewed-by: Hans de Goede --- diff --git a/src/filter.c b/src/filter.c index 7db78bae..4b2bbadd 100644 --- a/src/filter.c +++ b/src/filter.c @@ -335,6 +335,7 @@ pointer_accel_profile_smooth_simple(struct motion_filter *filter, double threshold = DEFAULT_THRESHOLD; /* units/ms */ double accel = DEFAULT_ACCELERATION; /* unitless factor */ double smooth_accel_coefficient; /* unitless factor */ + double factor; /* unitless factor */ if (threshold < 1.0) threshold = 1.0; @@ -349,12 +350,12 @@ pointer_accel_profile_smooth_simple(struct motion_filter *filter, if (velocity <= threshold) return 1.0; - velocity /= threshold; - if (velocity >= accel) + factor = velocity/threshold; + if (factor >= accel) return accel; - /* Velocity is between 1.0 and accel, scale this to 0.0 - 1.0 */ - velocity = (velocity - 1.0) / (accel - 1.0); - smooth_accel_coefficient = calc_penumbral_gradient(velocity); + /* factor is between 1.0 and accel, scale this to 0.0 - 1.0 */ + factor = (factor - 1.0) / (accel - 1.0); + smooth_accel_coefficient = calc_penumbral_gradient(factor); return 1.0 + (smooth_accel_coefficient * (accel - 1.0)); }