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.

diff --git a/src/config_directives.c b/src/config_directives.c
index 82e1a3461dc38e4c27eb4526817807de539e79c2..879d225e64e60796e61b104ab6a2257c28fc0462 100644 (file)
--- a/src/config_directives.c
+++ b/src/config_directives.c
@@ -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) {
     if (strcasecmp(modename, DEFAULT_BINDING_MODE) == 0) {
         ELOG("You cannot use the name %s for your mode\n", DEFAULT_BINDING_MODE);
-        exit(1);
+        return;
     }
 
     struct Mode *mode;
     SLIST_FOREACH(mode, &modes, modes) {
         if (strcmp(mode->name, modename) == 0) {
             ELOG("The binding mode with name \"%s\" is defined at least twice.\n", modename);
-            exit(1);
         }
     }