Don't exit() on redefined binding mode. (#2638)
Doing a hard exit() is a rather harsh action for something i3 can handle perfectly fine and is only meant to be a check to make debugging easier for users in certain situations.
This commit is contained in:
parent
d58dbc3a77
commit
7f84f49846
|
@ -124,14 +124,13 @@ CFGFUN(mode_binding, const char *bindtype, const char *modifiers, const char *ke
|
||||||
CFGFUN(enter_mode, const char *pango_markup, const char *modename) {
|
CFGFUN(enter_mode, const char *pango_markup, const char *modename) {
|
||||||
if (strcasecmp(modename, DEFAULT_BINDING_MODE) == 0) {
|
if (strcasecmp(modename, DEFAULT_BINDING_MODE) == 0) {
|
||||||
ELOG("You cannot use the name %s for your mode\n", DEFAULT_BINDING_MODE);
|
ELOG("You cannot use the name %s for your mode\n", DEFAULT_BINDING_MODE);
|
||||||
exit(1);
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
struct Mode *mode;
|
struct Mode *mode;
|
||||||
SLIST_FOREACH(mode, &modes, modes) {
|
SLIST_FOREACH(mode, &modes, modes) {
|
||||||
if (strcmp(mode->name, modename) == 0) {
|
if (strcmp(mode->name, modename) == 0) {
|
||||||
ELOG("The binding mode with name \"%s\" is defined at least twice.\n", modename);
|
ELOG("The binding mode with name \"%s\" is defined at least twice.\n", modename);
|
||||||
exit(1);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue