Restore pre-5.9.0-dev behavior of touch_use_crosshair=false shootline (#14389)

* Fix incorrect shootline after releasing pointer if touch_use_crosshair=false

This happened because Android reuses pointer IDs.
Also includes a refactor to merge "m_known_ids" and "m_pointer_pos".

* Restore pre-5.9.0-dev behavior of shootline when !m_has_move_id
This commit is contained in:
grorp 2024-02-24 13:12:53 +01:00 committed by GitHub
parent 492aab20fe
commit 57de599a29
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 17 additions and 34 deletions

@ -716,12 +716,9 @@ void TouchScreenGUI::handleReleaseEvent(size_t evt_id)
<< evt_id << std::endl; << evt_id << std::endl;
} }
for (auto iter = m_known_ids.begin(); iter != m_known_ids.end(); ++iter) { // By the way: Android reuses pointer IDs, so m_pointer_pos[evt_id]
if (iter->id == evt_id) { // will be overwritten soon anyway.
m_known_ids.erase(iter); m_pointer_pos.erase(evt_id);
break;
}
}
} }
void TouchScreenGUI::translateEvent(const SEvent &event) void TouchScreenGUI::translateEvent(const SEvent &event)
@ -748,17 +745,6 @@ void TouchScreenGUI::translateEvent(const SEvent &event)
const v2s32 dir_fixed = touch_pos - fixed_joystick_center; const v2s32 dir_fixed = touch_pos - fixed_joystick_center;
if (event.TouchInput.Event == ETIE_PRESSED_DOWN) { if (event.TouchInput.Event == ETIE_PRESSED_DOWN) {
/*
* Add to own copy of event list...
* android would provide this information but Irrlicht guys don't
* wanna design an efficient interface
*/
id_status to_be_added{};
to_be_added.id = event.TouchInput.ID;
to_be_added.X = event.TouchInput.X;
to_be_added.Y = event.TouchInput.Y;
m_known_ids.push_back(to_be_added);
size_t eventID = event.TouchInput.ID; size_t eventID = event.TouchInput.ID;
touch_gui_button_id button = getButtonID(X, Y); touch_gui_button_id button = getButtonID(X, Y);
@ -815,6 +801,7 @@ void TouchScreenGUI::translateEvent(const SEvent &event)
m_move_id = event.TouchInput.ID; m_move_id = event.TouchInput.ID;
m_move_has_really_moved = false; m_move_has_really_moved = false;
m_move_downtime = porting::getTimeMs(); m_move_downtime = porting::getTimeMs();
m_move_pos = touch_pos;
// DON'T reset m_tap_state here, otherwise many short taps // DON'T reset m_tap_state here, otherwise many short taps
// will be ignored if you tap very fast. // will be ignored if you tap very fast.
} }
@ -833,8 +820,7 @@ void TouchScreenGUI::translateEvent(const SEvent &event)
m_pointer_pos[event.TouchInput.ID] == touch_pos) m_pointer_pos[event.TouchInput.ID] == touch_pos)
return; return;
const v2s32 free_joystick_center = v2s32(m_pointer_pos[event.TouchInput.ID].X, const v2s32 free_joystick_center = m_pointer_pos[event.TouchInput.ID];
m_pointer_pos[event.TouchInput.ID].Y);
const v2s32 dir_free = touch_pos - free_joystick_center; const v2s32 dir_free = touch_pos - free_joystick_center;
const double touch_threshold_sq = m_touchscreen_threshold * m_touchscreen_threshold; const double touch_threshold_sq = m_touchscreen_threshold * m_touchscreen_threshold;
@ -843,6 +829,7 @@ void TouchScreenGUI::translateEvent(const SEvent &event)
if (dir_free.getLengthSQ() > touch_threshold_sq || m_move_has_really_moved) { if (dir_free.getLengthSQ() > touch_threshold_sq || m_move_has_really_moved) {
m_move_has_really_moved = true; m_move_has_really_moved = true;
m_move_pos = touch_pos;
m_pointer_pos[event.TouchInput.ID] = touch_pos; m_pointer_pos[event.TouchInput.ID] = touch_pos;
if (m_tap_state == TapState::None || m_draw_crosshair) { if (m_tap_state == TapState::None || m_draw_crosshair) {
@ -1023,7 +1010,9 @@ void TouchScreenGUI::step(float dtime)
// thus the camera position can change, it doesn't suffice to update the // thus the camera position can change, it doesn't suffice to update the
// shootline when a touch event occurs. // shootline when a touch event occurs.
// Note that the shootline isn't used if touch_use_crosshair is enabled. // Note that the shootline isn't used if touch_use_crosshair is enabled.
if (!m_draw_crosshair) { // Only updating when m_has_move_id means that the shootline will stay at
// it's last in-world position when the player doesn't need it.
if (!m_draw_crosshair && m_has_move_id) {
v2s32 pointer_pos = getPointerPos(); v2s32 pointer_pos = getPointerPos();
m_shootline = m_device m_shootline = m_device
->getSceneManager() ->getSceneManager()
@ -1061,8 +1050,8 @@ void TouchScreenGUI::setVisible(bool visible)
// clear all active buttons // clear all active buttons
if (!visible) { if (!visible) {
while (!m_known_ids.empty()) while (!m_pointer_pos.empty())
handleReleaseEvent(m_known_ids.begin()->id); handleReleaseEvent(m_pointer_pos.begin()->first);
m_settings_bar.hide(); m_settings_bar.hide();
m_rare_controls_bar.hide(); m_rare_controls_bar.hide();
@ -1092,7 +1081,9 @@ v2s32 TouchScreenGUI::getPointerPos()
{ {
if (m_draw_crosshair) if (m_draw_crosshair)
return v2s32(m_screensize.X / 2, m_screensize.Y / 2); return v2s32(m_screensize.X / 2, m_screensize.Y / 2);
return m_pointer_pos[m_move_id]; // We can't just use m_pointer_pos[m_move_id] because applyContextControls
// may emit release events after m_pointer_pos[m_move_id] is erased.
return m_move_pos;
} }
void TouchScreenGUI::emitMouseEvent(EMOUSE_INPUT_EVENT type) void TouchScreenGUI::emitMouseEvent(EMOUSE_INPUT_EVENT type)

@ -246,6 +246,8 @@ private:
size_t m_move_id; size_t m_move_id;
bool m_move_has_really_moved = false; bool m_move_has_really_moved = false;
u64 m_move_downtime = 0; u64 m_move_downtime = 0;
// m_move_pos stays valid even after m_move_id has been released.
v2s32 m_move_pos;
bool m_has_joystick_id = false; bool m_has_joystick_id = false;
size_t m_joystick_id; size_t m_joystick_id;
@ -281,16 +283,6 @@ private:
const rect<s32> &button_rect, int texture_id, const rect<s32> &button_rect, int texture_id,
bool visible = true); bool visible = true);
struct id_status
{
size_t id;
int X;
int Y;
};
// vector to store known ids and their initial touch positions
std::vector<id_status> m_known_ids;
// handle a button event // handle a button event
void handleButtonEvent(touch_gui_button_id bID, size_t eventID, bool action); void handleButtonEvent(touch_gui_button_id bID, size_t eventID, bool action);
@ -303,7 +295,7 @@ private:
// apply joystick status // apply joystick status
void applyJoystickStatus(); void applyJoystickStatus();
// array for saving last known position of a pointer // map to store the IDs and positions of currently pressed pointers
std::unordered_map<size_t, v2s32> m_pointer_pos; std::unordered_map<size_t, v2s32> m_pointer_pos;
// settings bar // settings bar