hid/gesture: Factor out last gesture retrieval into its own function
Deduplicates a commonly repeated expression.
This commit is contained in:
parent
a9d8e24e47
commit
44556dc21a
|
@ -154,8 +154,8 @@ void Controller_Gesture::UpdateGestureSharedMemory(u8* data, std::size_t size,
|
||||||
|
|
||||||
void Controller_Gesture::NewGesture(GestureProperties& gesture, TouchType& type,
|
void Controller_Gesture::NewGesture(GestureProperties& gesture, TouchType& type,
|
||||||
Attribute& attributes) {
|
Attribute& attributes) {
|
||||||
const auto& last_entry =
|
const auto& last_entry = GetLastGestureEntry();
|
||||||
shared_memory.gesture_states[(shared_memory.header.last_entry_index + 16) % 17];
|
|
||||||
gesture.detection_count++;
|
gesture.detection_count++;
|
||||||
type = TouchType::Touch;
|
type = TouchType::Touch;
|
||||||
|
|
||||||
|
@ -168,8 +168,7 @@ void Controller_Gesture::NewGesture(GestureProperties& gesture, TouchType& type,
|
||||||
|
|
||||||
void Controller_Gesture::UpdateExistingGesture(GestureProperties& gesture, TouchType& type,
|
void Controller_Gesture::UpdateExistingGesture(GestureProperties& gesture, TouchType& type,
|
||||||
f32 time_difference) {
|
f32 time_difference) {
|
||||||
const auto& last_entry =
|
const auto& last_entry = GetLastGestureEntry();
|
||||||
shared_memory.gesture_states[(shared_memory.header.last_entry_index + 16) % 17];
|
|
||||||
|
|
||||||
// Promote to pan type if touch moved
|
// Promote to pan type if touch moved
|
||||||
for (size_t id = 0; id < MAX_POINTS; id++) {
|
for (size_t id = 0; id < MAX_POINTS; id++) {
|
||||||
|
@ -204,8 +203,8 @@ void Controller_Gesture::UpdateExistingGesture(GestureProperties& gesture, Touch
|
||||||
void Controller_Gesture::EndGesture(GestureProperties& gesture,
|
void Controller_Gesture::EndGesture(GestureProperties& gesture,
|
||||||
GestureProperties& last_gesture_props, TouchType& type,
|
GestureProperties& last_gesture_props, TouchType& type,
|
||||||
Attribute& attributes, f32 time_difference) {
|
Attribute& attributes, f32 time_difference) {
|
||||||
const auto& last_entry =
|
const auto& last_entry = GetLastGestureEntry();
|
||||||
shared_memory.gesture_states[(shared_memory.header.last_entry_index + 16) % 17];
|
|
||||||
if (last_gesture_props.active_points != 0) {
|
if (last_gesture_props.active_points != 0) {
|
||||||
switch (last_entry.type) {
|
switch (last_entry.type) {
|
||||||
case TouchType::Touch:
|
case TouchType::Touch:
|
||||||
|
@ -255,10 +254,9 @@ void Controller_Gesture::UpdatePanEvent(GestureProperties& gesture,
|
||||||
GestureProperties& last_gesture_props, TouchType& type,
|
GestureProperties& last_gesture_props, TouchType& type,
|
||||||
f32 time_difference) {
|
f32 time_difference) {
|
||||||
auto& cur_entry = shared_memory.gesture_states[shared_memory.header.last_entry_index];
|
auto& cur_entry = shared_memory.gesture_states[shared_memory.header.last_entry_index];
|
||||||
const auto& last_entry =
|
const auto& last_entry = GetLastGestureEntry();
|
||||||
shared_memory.gesture_states[(shared_memory.header.last_entry_index + 16) % 17];
|
|
||||||
cur_entry.delta = gesture.mid_point - last_entry.pos;
|
|
||||||
|
|
||||||
|
cur_entry.delta = gesture.mid_point - last_entry.pos;
|
||||||
cur_entry.vel_x = static_cast<f32>(cur_entry.delta.x) / time_difference;
|
cur_entry.vel_x = static_cast<f32>(cur_entry.delta.x) / time_difference;
|
||||||
cur_entry.vel_y = static_cast<f32>(cur_entry.delta.y) / time_difference;
|
cur_entry.vel_y = static_cast<f32>(cur_entry.delta.y) / time_difference;
|
||||||
last_pan_time_difference = time_difference;
|
last_pan_time_difference = time_difference;
|
||||||
|
@ -284,8 +282,7 @@ void Controller_Gesture::EndPanEvent(GestureProperties& gesture,
|
||||||
GestureProperties& last_gesture_props, TouchType& type,
|
GestureProperties& last_gesture_props, TouchType& type,
|
||||||
f32 time_difference) {
|
f32 time_difference) {
|
||||||
auto& cur_entry = shared_memory.gesture_states[shared_memory.header.last_entry_index];
|
auto& cur_entry = shared_memory.gesture_states[shared_memory.header.last_entry_index];
|
||||||
const auto& last_entry =
|
const auto& last_entry = GetLastGestureEntry();
|
||||||
shared_memory.gesture_states[(shared_memory.header.last_entry_index + 16) % 17];
|
|
||||||
cur_entry.vel_x =
|
cur_entry.vel_x =
|
||||||
static_cast<f32>(last_entry.delta.x) / (last_pan_time_difference + time_difference);
|
static_cast<f32>(last_entry.delta.x) / (last_pan_time_difference + time_difference);
|
||||||
cur_entry.vel_y =
|
cur_entry.vel_y =
|
||||||
|
@ -309,8 +306,8 @@ void Controller_Gesture::EndPanEvent(GestureProperties& gesture,
|
||||||
void Controller_Gesture::SetSwipeEvent(GestureProperties& gesture,
|
void Controller_Gesture::SetSwipeEvent(GestureProperties& gesture,
|
||||||
GestureProperties& last_gesture_props, TouchType& type) {
|
GestureProperties& last_gesture_props, TouchType& type) {
|
||||||
auto& cur_entry = shared_memory.gesture_states[shared_memory.header.last_entry_index];
|
auto& cur_entry = shared_memory.gesture_states[shared_memory.header.last_entry_index];
|
||||||
const auto& last_entry =
|
const auto& last_entry = GetLastGestureEntry();
|
||||||
shared_memory.gesture_states[(shared_memory.header.last_entry_index + 16) % 17];
|
|
||||||
type = TouchType::Swipe;
|
type = TouchType::Swipe;
|
||||||
gesture = last_gesture_props;
|
gesture = last_gesture_props;
|
||||||
force_update = true;
|
force_update = true;
|
||||||
|
@ -353,6 +350,14 @@ std::optional<std::size_t> Controller_Gesture::GetUnusedFingerID() const {
|
||||||
return std::nullopt;
|
return std::nullopt;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Controller_Gesture::GestureState& Controller_Gesture::GetLastGestureEntry() {
|
||||||
|
return shared_memory.gesture_states[(shared_memory.header.last_entry_index + 16) % 17];
|
||||||
|
}
|
||||||
|
|
||||||
|
const Controller_Gesture::GestureState& Controller_Gesture::GetLastGestureEntry() const {
|
||||||
|
return shared_memory.gesture_states[(shared_memory.header.last_entry_index + 16) % 17];
|
||||||
|
}
|
||||||
|
|
||||||
std::size_t Controller_Gesture::UpdateTouchInputEvent(
|
std::size_t Controller_Gesture::UpdateTouchInputEvent(
|
||||||
const std::tuple<float, float, bool>& touch_input, std::size_t finger_id) {
|
const std::tuple<float, float, bool>& touch_input, std::size_t finger_id) {
|
||||||
const auto& [x, y, pressed] = touch_input;
|
const auto& [x, y, pressed] = touch_input;
|
||||||
|
|
|
@ -162,7 +162,11 @@ private:
|
||||||
TouchType& type);
|
TouchType& type);
|
||||||
|
|
||||||
// Returns an unused finger id, if there is no fingers available std::nullopt is returned.
|
// Returns an unused finger id, if there is no fingers available std::nullopt is returned.
|
||||||
std::optional<size_t> GetUnusedFingerID() const;
|
[[nodiscard]] std::optional<size_t> GetUnusedFingerID() const;
|
||||||
|
|
||||||
|
// Retrieves the last gesture entry, as indicated by shared memory indices.
|
||||||
|
[[nodiscard]] GestureState& GetLastGestureEntry();
|
||||||
|
[[nodiscard]] const GestureState& GetLastGestureEntry() const;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* If the touch is new it tries to assign a new finger id, if there is no fingers available no
|
* If the touch is new it tries to assign a new finger id, if there is no fingers available no
|
||||||
|
|
Reference in New Issue