diff --git a/crates/bevy_gilrs/src/converter.rs b/crates/bevy_gilrs/src/converter.rs index 35c7159a94b14..e5ce04aa9c302 100644 --- a/crates/bevy_gilrs/src/converter.rs +++ b/crates/bevy_gilrs/src/converter.rs @@ -37,8 +37,9 @@ pub fn convert_axis(axis: gilrs::Axis) -> Option { gilrs::Axis::RightStickX => Some(GamepadAxisType::RightStickX), gilrs::Axis::RightStickY => Some(GamepadAxisType::RightStickY), gilrs::Axis::RightZ => Some(GamepadAxisType::RightZ), - gilrs::Axis::DPadX => Some(GamepadAxisType::DPadX), - gilrs::Axis::DPadY => Some(GamepadAxisType::DPadY), - gilrs::Axis::Unknown => None, + // The `axis_dpad_to_button` gilrs filter should filter out all DPadX and DPadY events. If + // it doesn't then we probably need an entry added to the following repo and an update to + // GilRs to use the updated database: https://github.com/gabomdq/SDL_GameControllerDB + gilrs::Axis::Unknown | gilrs::Axis::DPadX | gilrs::Axis::DPadY => None, } } diff --git a/crates/bevy_gilrs/src/gilrs_system.rs b/crates/bevy_gilrs/src/gilrs_system.rs index 89922ee7f2d4a..6a058d05fd063 100644 --- a/crates/bevy_gilrs/src/gilrs_system.rs +++ b/crates/bevy_gilrs/src/gilrs_system.rs @@ -2,7 +2,7 @@ use crate::converter::{convert_axis, convert_button, convert_gamepad_id}; use bevy_ecs::event::EventWriter; use bevy_ecs::system::{NonSend, NonSendMut}; use bevy_input::{gamepad::GamepadEventRaw, prelude::*}; -use gilrs::{EventType, Gilrs}; +use gilrs::{ev::filter::axis_dpad_to_button, EventType, Filter, Gilrs}; pub fn gilrs_event_startup_system(gilrs: NonSend, mut events: EventWriter) { for (id, _) in gilrs.gamepads() { @@ -14,7 +14,12 @@ pub fn gilrs_event_startup_system(gilrs: NonSend, mut events: EventWriter } pub fn gilrs_event_system(mut gilrs: NonSendMut, mut events: EventWriter) { - while let Some(gilrs_event) = gilrs.next_event() { + while let Some(gilrs_event) = gilrs + .next_event() + .filter_ev(&axis_dpad_to_button, &mut gilrs) + { + gilrs.update(&gilrs_event); + match gilrs_event.event { EventType::Connected => { events.send(GamepadEventRaw::new( diff --git a/crates/bevy_input/src/gamepad.rs b/crates/bevy_input/src/gamepad.rs index 5fd06285a0b70..ee4488a1b1c43 100644 --- a/crates/bevy_input/src/gamepad.rs +++ b/crates/bevy_input/src/gamepad.rs @@ -135,8 +135,6 @@ pub enum GamepadAxisType { RightStickX, RightStickY, RightZ, - DPadX, - DPadY, } #[derive(Debug, Copy, Clone, PartialEq, Eq, Hash)] @@ -412,15 +410,13 @@ const ALL_BUTTON_TYPES: [GamepadButtonType; 19] = [ GamepadButtonType::DPadRight, ]; -const ALL_AXIS_TYPES: [GamepadAxisType; 8] = [ +const ALL_AXIS_TYPES: [GamepadAxisType; 6] = [ GamepadAxisType::LeftStickX, GamepadAxisType::LeftStickY, GamepadAxisType::LeftZ, GamepadAxisType::RightStickX, GamepadAxisType::RightStickY, GamepadAxisType::RightZ, - GamepadAxisType::DPadX, - GamepadAxisType::DPadY, ]; #[cfg(test)]