diff --git a/Source/Core/Core/HotkeyManager.cpp b/Source/Core/Core/HotkeyManager.cpp index db60003036..80b293f940 100644 --- a/Source/Core/Core/HotkeyManager.cpp +++ b/Source/Core/Core/HotkeyManager.cpp @@ -20,7 +20,7 @@ #include "InputCommon/GCPadStatus.h" // clang-format off -constexpr std::array hotkey_labels{{ +constexpr std::array s_hotkey_labels{{ _trans("Open"), _trans("Change Disc"), _trans("Eject Disc"), @@ -156,11 +156,11 @@ constexpr std::array hotkey_labels{{ _trans("Load State"), }}; // clang-format on -static_assert(NUM_HOTKEYS == hotkey_labels.size(), "Wrong count of hotkey_labels"); +static_assert(NUM_HOTKEYS == s_hotkey_labels.size(), "Wrong count of hotkey_labels"); namespace HotkeyManagerEmu { -static std::array s_hotkeyDown; +static std::array s_hotkey_down; static HotkeyStatus s_hotkey; static bool s_enabled; @@ -194,14 +194,14 @@ bool IsPressed(int id, bool held) static_cast(s_config.GetController(0))->GetIndexForGroup(group, id); if (s_hotkey.button[group] & (1 << group_key)) { - bool pressed = !!(s_hotkeyDown[group] & (1 << group_key)); - s_hotkeyDown[group] |= (1 << group_key); + const bool pressed = !!(s_hotkey_down[group] & (1 << group_key)); + s_hotkey_down[group] |= (1 << group_key); if (!pressed || held) return true; } else { - s_hotkeyDown[group] &= ~(1 << group_key); + s_hotkey_down[group] &= ~(1 << group_key); } return false; @@ -217,7 +217,7 @@ void Initialize() // load the saved controller config s_config.LoadConfig(true); - s_hotkeyDown = {}; + s_hotkey_down = {}; s_enabled = true; } @@ -245,7 +245,7 @@ struct HotkeyGroupInfo Hotkey last; }; -constexpr std::array groups_info = { +constexpr std::array s_groups_info = { {{_trans("General"), HK_OPEN, HK_EXIT}, {_trans("Volume"), HK_VOLUME_DOWN, HK_VOLUME_TOGGLE_MUTE}, {_trans("Emulation Speed"), HK_DECREASE_EMULATION_SPEED, HK_TOGGLE_THROTTLE}, @@ -271,12 +271,12 @@ HotkeyManager::HotkeyManager() for (std::size_t group = 0; group < m_hotkey_groups.size(); group++) { m_hotkey_groups[group] = - (m_keys[group] = new ControllerEmu::Buttons("Keys", groups_info[group].name)); + (m_keys[group] = new ControllerEmu::Buttons("Keys", s_groups_info[group].name)); groups.emplace_back(m_hotkey_groups[group]); - for (int key = groups_info[group].first; key <= groups_info[group].last; key++) + for (int key = s_groups_info[group].first; key <= s_groups_info[group].last; key++) { m_keys[group]->controls.emplace_back( - new ControllerEmu::Input(ControllerEmu::Translate, hotkey_labels[key])); + new ControllerEmu::Input(ControllerEmu::Translate, s_hotkey_labels[key])); } } } @@ -293,9 +293,9 @@ std::string HotkeyManager::GetName() const void HotkeyManager::GetInput(HotkeyStatus* const kb) { const auto lock = GetStateLock(); - for (std::size_t group = 0; group < groups_info.size(); group++) + for (std::size_t group = 0; group < s_groups_info.size(); group++) { - const int group_count = (groups_info[group].last - groups_info[group].first) + 1; + const int group_count = (s_groups_info[group].last - s_groups_info[group].first) + 1; std::vector bitmasks(group_count); for (size_t key = 0; key < bitmasks.size(); key++) bitmasks[key] = static_cast(1 << key); @@ -312,15 +312,15 @@ ControllerEmu::ControlGroup* HotkeyManager::GetHotkeyGroup(HotkeyGroup group) co int HotkeyManager::FindGroupByID(int id) const { - const auto i = std::find_if(groups_info.begin(), groups_info.end(), + const auto i = std::find_if(s_groups_info.begin(), s_groups_info.end(), [id](const auto& entry) { return entry.last >= id; }); - return static_cast(std::distance(groups_info.begin(), i)); + return static_cast(std::distance(s_groups_info.begin(), i)); } int HotkeyManager::GetIndexForGroup(int group, int id) const { - return id - groups_info[group].first; + return id - s_groups_info[group].first; } void HotkeyManager::LoadDefaults(const ControllerInterface& ciface)