Merge pull request #52693 from akien-mga/fixup-inputmap-macos-override

InputMap: Fixup `macos` feature tag overrides after #52291
This commit is contained in:
Rémi Verschelde 2021-09-15 13:20:40 +02:00 committed by GitHub
commit fbef32ab46
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -703,11 +703,11 @@ void InputMap::load_default() {
OrderedHashMap<String, List<Ref<InputEvent>>> builtins = get_builtins();
// List of Builtins which have an override for macOS.
Vector<String> osx_builtins;
Vector<String> macos_builtins;
for (OrderedHashMap<String, List<Ref<InputEvent>>>::Element E = builtins.front(); E; E = E.next()) {
if (String(E.key()).ends_with(".macos")) {
// Strip .macos from name: some_input_name.macos -> some_input_name
osx_builtins.push_back(String(E.key()).split(".")[0]);
macos_builtins.push_back(String(E.key()).split(".")[0]);
}
}
@ -717,12 +717,12 @@ void InputMap::load_default() {
String override_for = fullname.split(".").size() > 1 ? fullname.split(".")[1] : "";
#ifdef APPLE_STYLE_KEYS
if (osx_builtins.has(name) && override_for != "osx") {
// Name has `osx` builtin but this particular one is for non-macOS systems - so skip.
if (macos_builtins.has(name) && override_for != "macos") {
// Name has `macos` builtin but this particular one is for non-macOS systems - so skip.
continue;
}
#else
if (override_for == "osx") {
if (override_for == "macos") {
// Override for macOS - not needed on non-macOS platforms.
continue;
}