From 3d96d7d9fb84bbf8c63fd6bda48093271794f218 Mon Sep 17 00:00:00 2001 From: Marcel Admiraal Date: Thu, 13 Jan 2022 18:00:25 +0000 Subject: [PATCH] Fix mouse speed not changing fast enough - Uses all accumulated movements when calculating speed - Discards old accumulated movements - Sets last mouse speed to zero when there is no movement --- core/os/input.h | 2 +- doc/classes/Input.xml | 4 ++-- main/input_default.cpp | 30 ++++++++++++++++++------------ main/input_default.h | 2 +- 4 files changed, 22 insertions(+), 16 deletions(-) diff --git a/core/os/input.h b/core/os/input.h index ed12ec654bb..ce4ee5bb668 100644 --- a/core/os/input.h +++ b/core/os/input.h @@ -107,7 +107,7 @@ public: virtual void vibrate_handheld(int p_duration_ms = 500) = 0; virtual Point2 get_mouse_position() const = 0; - virtual Point2 get_last_mouse_speed() const = 0; + virtual Point2 get_last_mouse_speed() = 0; virtual int get_mouse_button_mask() const = 0; virtual void warp_mouse_position(const Vector2 &p_to) = 0; diff --git a/doc/classes/Input.xml b/doc/classes/Input.xml index 73e4557858a..635b1401dec 100644 --- a/doc/classes/Input.xml +++ b/doc/classes/Input.xml @@ -169,10 +169,10 @@ Returns the strength of the joypad vibration: x is the strength of the weak motor, and y is the strength of the strong motor. - + - Returns the mouse speed for the last time the cursor was moved, and this until the next frame where the mouse moves. This means that even if the mouse is not moving, this function will still return the value of the last motion. + Returns the last mouse speed. To provide a precise and jitter-free speed, mouse speed is only calculated every 0.1s. Therefore, mouse speed will lag mouse movements. diff --git a/main/input_default.cpp b/main/input_default.cpp index 5c284dafb81..2e3453a09b5 100644 --- a/main/input_default.cpp +++ b/main/input_default.cpp @@ -42,32 +42,37 @@ void InputDefault::SpeedTrack::update(const Vector2 &p_delta_p) { float delta_t = tdiff / 1000000.0; last_tick = tick; + if (delta_t > max_ref_frame) { + // First movement in a long time, reset and start again. + speed = Vector2(); + accum = p_delta_p; + accum_t = 0; + return; + } + accum += p_delta_p; accum_t += delta_t; - if (accum_t > max_ref_frame * 10) { - accum_t = max_ref_frame * 10; + if (accum_t < min_ref_frame) { + // Not enough time has passed to calculate speed precisely. + return; } - while (accum_t >= min_ref_frame) { - float slice_t = min_ref_frame / accum_t; - Vector2 slice = accum * slice_t; - accum = accum - slice; - accum_t -= min_ref_frame; - - speed = (slice / min_ref_frame).linear_interpolate(speed, min_ref_frame / max_ref_frame); - } + speed = accum / accum_t; + accum = Vector2(); + accum_t = 0; } void InputDefault::SpeedTrack::reset() { last_tick = OS::get_singleton()->get_ticks_usec(); speed = Vector2(); + accum = Vector2(); accum_t = 0; } InputDefault::SpeedTrack::SpeedTrack() { min_ref_frame = 0.1; - max_ref_frame = 0.3; + max_ref_frame = 3.0; reset(); } @@ -559,7 +564,8 @@ void InputDefault::set_mouse_position(const Point2 &p_posf) { Point2 InputDefault::get_mouse_position() const { return mouse_pos; } -Point2 InputDefault::get_last_mouse_speed() const { +Point2 InputDefault::get_last_mouse_speed() { + mouse_speed_track.update(Vector2()); return mouse_speed_track.speed; } diff --git a/main/input_default.h b/main/input_default.h index 2428cf72c5f..11b42b8c0b8 100644 --- a/main/input_default.h +++ b/main/input_default.h @@ -244,7 +244,7 @@ public: virtual Vector3 get_gyroscope() const; virtual Point2 get_mouse_position() const; - virtual Point2 get_last_mouse_speed() const; + virtual Point2 get_last_mouse_speed(); virtual int get_mouse_button_mask() const; virtual void warp_mouse_position(const Vector2 &p_to);