no-op change: move bind->release check into loop

We don’t have to negate the check anymore, making it more readable.
This commit is contained in:
Michael Stapelberg 2017-05-24 20:41:17 +02:00
parent 6d9b165fb4
commit 0acd11a8d7
1 changed files with 2 additions and 4 deletions

View File

@ -268,14 +268,12 @@ static Binding *get_binding(i3_event_state_mask_t state_filtered, bool is_releas
const bool mods_match = modifiers_match(modifiers_mask, modifiers_state); const bool mods_match = modifiers_match(modifiers_mask, modifiers_state);
DLOG("binding_keycode->modifiers = %d, modifiers_mask = %d, modifiers_state = %d, mods_match = %s\n", DLOG("binding_keycode->modifiers = %d, modifiers_mask = %d, modifiers_state = %d, mods_match = %s\n",
binding_keycode->modifiers, modifiers_mask, modifiers_state, (mods_match ? "yes" : "no")); binding_keycode->modifiers, modifiers_mask, modifiers_state, (mods_match ? "yes" : "no"));
if (mods_match) { if (mods_match || (bind->release == B_UPON_KEYRELEASE_IGNORE_MODS && is_release)) {
found_keycode = true; found_keycode = true;
break; break;
} }
} }
if (!found_keycode && if (!found_keycode) {
(bind->release != B_UPON_KEYRELEASE_IGNORE_MODS ||
!is_release)) {
continue; continue;
} }
} }