From 26f17a172353098740d97cce531f4c7c06abd4e4 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Sat, 25 Feb 2017 00:35:02 -0500 Subject: ControlGroup: Convert group type enum into an enum class Gets some constants out of the ControllerEmu namespace, and modifies ControlGroup so that it uses the enum type itself to represent the underlying type, rather than a u32 value. --- .../ControllerEmu/ControlGroup/AnalogStick.cpp | 2 +- .../ControllerEmu/ControlGroup/Buttons.cpp | 2 +- .../ControllerEmu/ControlGroup/ControlGroup.cpp | 9 ++++--- .../ControllerEmu/ControlGroup/ControlGroup.h | 29 +++++++++++----------- .../ControllerEmu/ControlGroup/Cursor.cpp | 2 +- .../ControllerEmu/ControlGroup/Extension.cpp | 2 +- .../ControllerEmu/ControlGroup/Force.cpp | 2 +- .../ControllerEmu/ControlGroup/MixedTriggers.cpp | 2 +- .../ControllerEmu/ControlGroup/Slider.cpp | 2 +- .../ControllerEmu/ControlGroup/Tilt.cpp | 2 +- .../ControllerEmu/ControlGroup/Triggers.cpp | 2 +- .../InputCommon/ControllerEmu/ControllerEmu.cpp | 4 +-- 12 files changed, 31 insertions(+), 29 deletions(-) (limited to 'Source/Core/InputCommon') diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/AnalogStick.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/AnalogStick.cpp index 8ee7357963..f8e2fabd59 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/AnalogStick.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/AnalogStick.cpp @@ -24,7 +24,7 @@ AnalogStick::AnalogStick(const char* const name_, ControlState default_radius) AnalogStick::AnalogStick(const char* const name_, const char* const ui_name_, ControlState default_radius) - : ControlGroup(name_, ui_name_, GROUP_TYPE_STICK) + : ControlGroup(name_, ui_name_, GroupType::Stick) { for (auto& named_direction : named_directions) controls.emplace_back(std::make_unique(named_direction)); diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Buttons.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Buttons.cpp index ef3ca79a0f..5a86839380 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Buttons.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Buttons.cpp @@ -16,7 +16,7 @@ Buttons::Buttons(const std::string& name_) : Buttons(name_, name_) } Buttons::Buttons(const std::string& ini_name, const std::string& group_name) - : ControlGroup(ini_name, group_name, GROUP_TYPE_BUTTONS) + : ControlGroup(ini_name, group_name, GroupType::Buttons) { numeric_settings.emplace_back(std::make_unique(_trans("Threshold"), 0.5)); } diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp index cd730697a5..0b8b7f19d9 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp @@ -14,12 +14,13 @@ namespace ControllerEmu { -ControlGroup::ControlGroup(const std::string& name_, const u32 type_) +ControlGroup::ControlGroup(const std::string& name_, const GroupType type_) : name(name_), ui_name(name_), type(type_) { } -ControlGroup::ControlGroup(const std::string& name_, const std::string& ui_name_, const u32 type_) +ControlGroup::ControlGroup(const std::string& name_, const std::string& ui_name_, + const GroupType type_) : name(name_), ui_name(ui_name_), type(type_) { } @@ -62,7 +63,7 @@ void ControlGroup::LoadConfig(IniFile::Section* sec, const std::string& defdev, } // extensions - if (type == GROUP_TYPE_EXTENSION) + if (type == GroupType::Extension) { Extension* const ext = (Extension*)this; @@ -115,7 +116,7 @@ void ControlGroup::SaveConfig(IniFile::Section* sec, const std::string& defdev, } // extensions - if (type == GROUP_TYPE_EXTENSION) + if (type == GroupType::Extension) { Extension* const ext = (Extension*)this; sec->Set(base + name, ext->attachments[ext->switch_extension]->GetName(), "None"); diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.h b/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.h index d1870f7f03..212205f333 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.h +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.h @@ -17,18 +17,18 @@ namespace ControllerEmu { class Control; -enum +enum class GroupType { - GROUP_TYPE_OTHER, - GROUP_TYPE_STICK, - GROUP_TYPE_MIXED_TRIGGERS, - GROUP_TYPE_BUTTONS, - GROUP_TYPE_FORCE, - GROUP_TYPE_EXTENSION, - GROUP_TYPE_TILT, - GROUP_TYPE_CURSOR, - GROUP_TYPE_TRIGGERS, - GROUP_TYPE_SLIDER + Other, + Stick, + MixedTriggers, + Buttons, + Force, + Extension, + Tilt, + Cursor, + Triggers, + Slider }; class ControlGroup @@ -102,8 +102,9 @@ public: } }; - explicit ControlGroup(const std::string& name, u32 type = GROUP_TYPE_OTHER); - ControlGroup(const std::string& name, const std::string& ui_name, u32 type = GROUP_TYPE_OTHER); + explicit ControlGroup(const std::string& name, GroupType type = GroupType::Other); + ControlGroup(const std::string& name, const std::string& ui_name, + GroupType type = GroupType::Other); virtual ~ControlGroup(); virtual void LoadConfig(IniFile::Section* sec, const std::string& defdev = "", @@ -115,7 +116,7 @@ public: const std::string name; const std::string ui_name; - const u32 type; + const GroupType type; std::vector> controls; std::vector> numeric_settings; diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Cursor.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Cursor.cpp index 1d0c55a8d5..bede6f1899 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Cursor.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Cursor.cpp @@ -19,7 +19,7 @@ namespace ControllerEmu { -Cursor::Cursor(const std::string& name_) : ControlGroup(name_, GROUP_TYPE_CURSOR) +Cursor::Cursor(const std::string& name_) : ControlGroup(name_, GroupType::Cursor) { for (auto& named_direction : named_directions) controls.emplace_back(std::make_unique(named_direction)); diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Extension.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Extension.cpp index b35eb40bbd..eb9dbc8a99 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Extension.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Extension.cpp @@ -10,7 +10,7 @@ namespace ControllerEmu { -Extension::Extension(const std::string& name_) : ControlGroup(name_, GROUP_TYPE_EXTENSION) +Extension::Extension(const std::string& name_) : ControlGroup(name_, GroupType::Extension) { } } // namespace ControllerEmu diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Force.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Force.cpp index fba3fb6097..43199810cb 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Force.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Force.cpp @@ -16,7 +16,7 @@ namespace ControllerEmu { -Force::Force(const std::string& name_) : ControlGroup(name_, GROUP_TYPE_FORCE) +Force::Force(const std::string& name_) : ControlGroup(name_, GroupType::Force) { controls.emplace_back(std::make_unique(_trans("Up"))); controls.emplace_back(std::make_unique(_trans("Down"))); diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/MixedTriggers.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/MixedTriggers.cpp index db8efc48a5..4a35f8047a 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/MixedTriggers.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/MixedTriggers.cpp @@ -17,7 +17,7 @@ namespace ControllerEmu { MixedTriggers::MixedTriggers(const std::string& name_) - : ControlGroup(name_, GROUP_TYPE_MIXED_TRIGGERS) + : ControlGroup(name_, GroupType::MixedTriggers) { numeric_settings.emplace_back(std::make_unique(_trans("Threshold"), 0.9)); } diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Slider.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Slider.cpp index 5fdf349bba..c662ffc4ae 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Slider.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Slider.cpp @@ -16,7 +16,7 @@ namespace ControllerEmu { -Slider::Slider(const std::string& name_) : ControlGroup(name_, GROUP_TYPE_SLIDER) +Slider::Slider(const std::string& name_) : ControlGroup(name_, GroupType::Slider) { controls.emplace_back(std::make_unique("Left")); controls.emplace_back(std::make_unique("Right")); diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Tilt.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Tilt.cpp index 1aaeab15e0..1c124c2547 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Tilt.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Tilt.cpp @@ -16,7 +16,7 @@ namespace ControllerEmu { -Tilt::Tilt(const std::string& name_) : ControlGroup(name_, GROUP_TYPE_TILT) +Tilt::Tilt(const std::string& name_) : ControlGroup(name_, GroupType::Tilt) { controls.emplace_back(std::make_unique("Forward")); controls.emplace_back(std::make_unique("Backward")); diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Triggers.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Triggers.cpp index 3e13b0b504..4680d4715b 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/Triggers.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/Triggers.cpp @@ -15,7 +15,7 @@ namespace ControllerEmu { -Triggers::Triggers(const std::string& name_) : ControlGroup(name_, GROUP_TYPE_TRIGGERS) +Triggers::Triggers(const std::string& name_) : ControlGroup(name_, GroupType::Triggers) { numeric_settings.emplace_back(std::make_unique(_trans("Dead Zone"), 0, 0, 50)); } diff --git a/Source/Core/InputCommon/ControllerEmu/ControllerEmu.cpp b/Source/Core/InputCommon/ControllerEmu/ControllerEmu.cpp index ce09966d2d..6de5042aaf 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControllerEmu.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControllerEmu.cpp @@ -39,7 +39,7 @@ void EmulatedController::UpdateReferences(const ControllerInterface& devi) control->control_ref.get()->UpdateReference(devi, default_device); // extension - if (ctrlGroup->type == GROUP_TYPE_EXTENSION) + if (ctrlGroup->type == GroupType::Extension) { for (auto& attachment : ((Extension*)ctrlGroup.get())->attachments) attachment->UpdateReferences(devi); @@ -52,7 +52,7 @@ void EmulatedController::UpdateDefaultDevice() for (auto& ctrlGroup : groups) { // extension - if (ctrlGroup->type == GROUP_TYPE_EXTENSION) + if (ctrlGroup->type == GroupType::Extension) { for (auto& ai : ((Extension*)ctrlGroup.get())->attachments) { -- cgit v1.2.3