fix: simplify key translation logic by removing clamping

This commit is contained in:
Nick Bolton 2026-07-06 11:08:35 +01:00
parent fc44b0bb18
commit 3f2571dfa3

View file

@ -455,11 +455,9 @@ KeyID ServerProxy::translateKey(KeyID id) const
} }
if (id2 != kKeyModifierIDNull) { if (id2 != kKeyModifierIDNull) {
return std::clamp<KeyModifierMask>( return s_translationTable[m_modifierTranslationTable[id2]][side];
s_translationTable[m_modifierTranslationTable[id2]][side], 0, kKeyModifierIDLast - 1
);
} else { } else {
return std::clamp<KeyModifierMask>(id, 0, kKeyModifierIDLast - 1); return id;
} }
} }
@ -489,7 +487,7 @@ KeyModifierMask ServerProxy::translateModifierMask(KeyModifierMask mask) const
if ((mask & KeyModifierSuper) != 0) { if ((mask & KeyModifierSuper) != 0) {
newMask |= s_masks[m_modifierTranslationTable[kKeyModifierIDSuper]]; newMask |= s_masks[m_modifierTranslationTable[kKeyModifierIDSuper]];
} }
return std::clamp<KeyModifierMask>(newMask, 0, kKeyModifierIDLast - 1); return newMask;
} }
void ServerProxy::enter() void ServerProxy::enter()