]> git.sur5r.net Git - i3/i3/blobdiff - src/commands.c
format **/*.c with clang-format-3.5
[i3/i3] / src / commands.c
index e1aa063b541f8f0a5f66c85bc9dce8b05eb47d89..03bb10a622be654530d1f7fae4cca1b2f14b4ca1 100644 (file)
 #include "shmlog.h"
 
 // Macros to make the YAJL API a bit easier to use.
-#define y(x, ...) yajl_gen_ ## x (cmd_output->json_gen, ##__VA_ARGS__)
-#define ystr(str) yajl_gen_string(cmd_output->json_gen, (unsigned char*)str, strlen(str))
-#define ysuccess(success) do { \
-    y(map_open); \
-    ystr("success"); \
-    y(bool, success); \
-    y(map_close); \
-} while (0)
-#define yerror(message) do { \
-    y(map_open); \
-    ystr("success"); \
-    y(bool, false); \
-    ystr("error"); \
-    ystr(message); \
-    y(map_close); \
-} while (0)
+#define y(x, ...) (cmd_output->json_gen != NULL ? yajl_gen_##x(cmd_output->json_gen, ##__VA_ARGS__) : 0)
+#define ystr(str) (cmd_output->json_gen != NULL ? yajl_gen_string(cmd_output->json_gen, (unsigned char *)str, strlen(str)) : 0)
+#define ysuccess(success)                   \
+    do {                                    \
+        if (cmd_output->json_gen != NULL) { \
+            y(map_open);                    \
+            ystr("success");                \
+            y(bool, success);               \
+            y(map_close);                   \
+        }                                   \
+    } while (0)
+#define yerror(message)                     \
+    do {                                    \
+        if (cmd_output->json_gen != NULL) { \
+            y(map_open);                    \
+            ystr("success");                \
+            y(bool, false);                 \
+            ystr("error");                  \
+            ystr(message);                  \
+            y(map_close);                   \
+        }                                   \
+    } while (0)
 
 /** When the command did not include match criteria (!), we use the currently
  * focused container. Do not confuse this case with a command which included
  * criteria but which did not match any windows. This macro has to be called in
  * every command.
  */
-#define HANDLE_EMPTY_MATCH do { \
-    if (match_is_empty(current_match)) { \
-        owindow *ow = smalloc(sizeof(owindow)); \
-        ow->con = focused; \
-        TAILQ_INIT(&owindows); \
-        TAILQ_INSERT_TAIL(&owindows, ow, owindows); \
-    } \
-} while (0)
-
+#define HANDLE_EMPTY_MATCH                              \
+    do {                                                \
+        if (match_is_empty(current_match)) {            \
+            owindow *ow = smalloc(sizeof(owindow));     \
+            ow->con = focused;                          \
+            TAILQ_INIT(&owindows);                      \
+            TAILQ_INSERT_TAIL(&owindows, ow, owindows); \
+        }                                               \
+    } while (0)
 
 /*
  * Returns true if a is definitely greater than b (using the given epsilon)
  *
  */
 static bool definitelyGreaterThan(float a, float b, float epsilon) {
-    return (a - b) > ( (fabs(a) < fabs(b) ? fabs(b) : fabs(a)) * epsilon);
+    return (a - b) > ((fabs(a) < fabs(b) ? fabs(b) : fabs(a)) * epsilon);
 }
 
 /*
@@ -72,7 +78,8 @@ static Output *get_output_from_string(Output *current_output, const char *output
         output = get_output_next_wrap(D_UP, current_output);
     else if (strcasecmp(output_str, "down") == 0)
         output = get_output_next_wrap(D_DOWN, current_output);
-    else output = get_output_by_name(output_str);
+    else
+        output = get_output_by_name(output_str);
 
     return output;
 }
@@ -95,7 +102,7 @@ static Output *get_output_of_con(Con *con) {
  * and return true, signaling that no further workspace switching should occur in the calling function.
  *
  */
-static bool maybe_back_and_forth(struct CommandResult *cmd_output, char *name) {
+static bool maybe_back_and_forth(struct CommandResultIR *cmd_output, char *name) {
     Con *ws = con_get_workspace(focused);
 
     /* If we switched to a different workspace, do nothing */
@@ -266,7 +273,7 @@ void cmd_criteria_init(I3_CMD) {
     }
     TAILQ_INIT(&owindows);
     /* copy all_cons */
-    TAILQ_FOREACH(con, &all_cons, all_cons) {
+    TAILQ_FOREACH (con, &all_cons, all_cons) {
         ow = smalloc(sizeof(owindow));
         ow->con = con;
         TAILQ_INSERT_TAIL(&owindows, ow, owindows);
@@ -317,7 +324,7 @@ void cmd_criteria_match_windows(I3_CMD) {
         }
     }
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
     }
 }
@@ -354,8 +361,8 @@ void cmd_criteria_add(I3_CMD, char *ctype, char *cvalue) {
             (end && *end != '\0')) {
             ELOG("Could not parse con id \"%s\"\n", cvalue);
         } else {
-            current_match->con_id = (Con*)parsed;
-            printf("id as int = %p\n", current_match->con_id);
+            current_match->con_id = (Con *)parsed;
+            DLOG("id as int = %p\n", current_match->con_id);
         }
         return;
     }
@@ -370,7 +377,7 @@ void cmd_criteria_add(I3_CMD, char *ctype, char *cvalue) {
             ELOG("Could not parse window id \"%s\"\n", cvalue);
         } else {
             current_match->id = parsed;
-            printf("window id as int = %d\n", current_match->id);
+            DLOG("window id as int = %d\n", current_match->id);
         }
         return;
     }
@@ -416,7 +423,7 @@ void cmd_move_con_to_workspace(I3_CMD, char *which) {
      *  when criteria wasn't specified and we don't have any window focused. */
     if ((!match_is_empty(current_match) && TAILQ_EMPTY(&owindows)) ||
         (match_is_empty(current_match) && focused->type == CT_WORKSPACE &&
-        !con_has_children(focused))) {
+         !con_has_children(focused))) {
         ysuccess(false);
         return;
     }
@@ -441,7 +448,7 @@ void cmd_move_con_to_workspace(I3_CMD, char *which) {
         return;
     }
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         con_move_to_workspace(current->con, ws, true, false);
     }
@@ -468,7 +475,7 @@ void cmd_move_con_to_workspace_back_and_forth(I3_CMD) {
 
     HANDLE_EMPTY_MATCH;
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         con_move_to_workspace(current->con, ws, true, false);
     }
@@ -483,8 +490,8 @@ void cmd_move_con_to_workspace_back_and_forth(I3_CMD) {
  *
  */
 void cmd_move_con_to_workspace_name(I3_CMD, char *name) {
-    if (strncasecmp(name, "__i3_", strlen("__i3_")) == 0) {
-        LOG("You cannot switch to the i3 internal workspaces.\n");
+    if (strncasecmp(name, "__", strlen("__")) == 0) {
+        LOG("You cannot move containers to i3-internal workspaces (\"%s\").\n", name);
         ysuccess(false);
         return;
     }
@@ -498,9 +505,8 @@ void cmd_move_con_to_workspace_name(I3_CMD, char *name) {
         ELOG("No windows match your criteria, cannot move.\n");
         ysuccess(false);
         return;
-    }
-    else if (match_is_empty(current_match) && focused->type == CT_WORKSPACE &&
-        !con_has_children(focused)) {
+    } else if (match_is_empty(current_match) && focused->type == CT_WORKSPACE &&
+               !con_has_children(focused)) {
         ysuccess(false);
         return;
     }
@@ -513,7 +519,7 @@ void cmd_move_con_to_workspace_name(I3_CMD, char *name) {
 
     HANDLE_EMPTY_MATCH;
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         con_move_to_workspace(current->con, ws, true, false);
     }
@@ -535,7 +541,7 @@ void cmd_move_con_to_workspace_number(I3_CMD, char *which) {
      *  when criteria wasn't specified and we don't have any window focused. */
     if ((!match_is_empty(current_match) && TAILQ_EMPTY(&owindows)) ||
         (match_is_empty(current_match) && focused->type == CT_WORKSPACE &&
-        !con_has_children(focused))) {
+         !con_has_children(focused))) {
         ysuccess(false);
         return;
     }
@@ -556,9 +562,9 @@ void cmd_move_con_to_workspace_number(I3_CMD, char *which) {
         return;
     }
 
-    TAILQ_FOREACH(output, &(croot->nodes_head), nodes)
+    TAILQ_FOREACH (output, &(croot->nodes_head), nodes)
         GREP_FIRST(workspace, output_get_content(output),
-            child->num == parsed_num);
+                   child->num == parsed_num);
 
     if (!workspace) {
         workspace = workspace_get(which, NULL);
@@ -568,7 +574,7 @@ void cmd_move_con_to_workspace_number(I3_CMD, char *which) {
 
     HANDLE_EMPTY_MATCH;
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         con_move_to_workspace(current->con, workspace, true, false);
     }
@@ -722,7 +728,7 @@ static bool cmd_resize_tiling_width_height(I3_CMD, Con *current, char *way, char
 
     /* Ensure all the other children have a percentage set. */
     Con *child;
-    TAILQ_FOREACH(child, &(current->parent->nodes_head), nodes) {
+    TAILQ_FOREACH (child, &(current->parent->nodes_head), nodes) {
         LOG("child->percent = %f (child %p)\n", child->percent, child);
         if (child->percent == 0.0)
             child->percent = percentage;
@@ -734,7 +740,7 @@ static bool cmd_resize_tiling_width_height(I3_CMD, Con *current, char *way, char
     LOG("subtract_percent = %f\n", subtract_percent);
     /* Ensure that the new percentages are positive and greater than
      * 0.05 to have a reasonable minimum size. */
-    TAILQ_FOREACH(child, &(current->parent->nodes_head), nodes) {
+    TAILQ_FOREACH (child, &(current->parent->nodes_head), nodes) {
         if (child == current)
             continue;
         if (!definitelyGreaterThan(child->percent - subtract_percent, 0.05, DBL_EPSILON)) {
@@ -752,7 +758,7 @@ static bool cmd_resize_tiling_width_height(I3_CMD, Con *current, char *way, char
     current->percent += ((double)ppt / 100.0);
     LOG("current->percent after = %f\n", current->percent);
 
-    TAILQ_FOREACH(child, &(current->parent->nodes_head), nodes) {
+    TAILQ_FOREACH (child, &(current->parent->nodes_head), nodes) {
         if (child == current)
             continue;
         child->percent -= subtract_percent;
@@ -780,7 +786,7 @@ void cmd_resize(I3_CMD, char *way, char *direction, char *resize_px, char *resiz
     HANDLE_EMPTY_MATCH;
 
     owindow *current;
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         /* Don't handle dock windows (issue #1201) */
         if (current->con->window && current->con->window->dock) {
             DLOG("This is a dock window. Not resizing (con = %p)\n)", current->con);
@@ -811,13 +817,13 @@ void cmd_resize(I3_CMD, char *way, char *direction, char *resize_px, char *resiz
  * Implementation of 'border normal|none|1pixel|toggle|pixel'.
  *
  */
-void cmd_border(I3_CMD, char *border_style_str, char *border_width ) {
+void cmd_border(I3_CMD, char *border_style_str, char *border_width) {
     DLOG("border style should be changed to %s with border width %s\n", border_style_str, border_width);
     owindow *current;
 
     HANDLE_EMPTY_MATCH;
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         int border_style = current->con->border_style;
         char *end;
@@ -841,7 +847,7 @@ void cmd_border(I3_CMD, char *border_style_str, char *border_width ) {
                 border_style = BS_NORMAL;
             else if (strcmp(border_style_str, "pixel") == 0)
                 border_style = BS_PIXEL;
-            else if (strcmp(border_style_str, "1pixel") == 0){
+            else if (strcmp(border_style_str, "1pixel") == 0) {
                 border_style = BS_PIXEL;
                 tmp_border_width = 1;
             } else if (strcmp(border_style_str, "none") == 0)
@@ -961,9 +967,9 @@ void cmd_workspace_number(I3_CMD, char *which) {
         return;
     }
 
-    TAILQ_FOREACH(output, &(croot->nodes_head), nodes)
+    TAILQ_FOREACH (output, &(croot->nodes_head), nodes)
         GREP_FIRST(workspace, output_get_content(output),
-            child->num == parsed_num);
+                   child->num == parsed_num);
 
     if (!workspace) {
         LOG("There is no workspace with number %ld, creating a new one.\n", parsed_num);
@@ -998,15 +1004,15 @@ void cmd_workspace_back_and_forth(I3_CMD) {
  *
  */
 void cmd_workspace_name(I3_CMD, char *name) {
-    if (strncasecmp(name, "__i3_", strlen("__i3_")) == 0) {
-        LOG("You cannot switch to the i3 internal workspaces.\n");
+    if (strncasecmp(name, "__", strlen("__")) == 0) {
+        LOG("You cannot switch to the i3-internal workspaces (\"%s\").\n", name);
         ysuccess(false);
         return;
     }
 
     DLOG("should switch to workspace %s\n", name);
     if (maybe_back_and_forth(cmd_output, name))
-       return;
+        return;
     workspace_show_by_name(name);
 
     cmd_output->needs_tree_render = true;
@@ -1022,7 +1028,7 @@ void cmd_mark(I3_CMD, char *mark) {
     DLOG("Clearing all windows which have that mark first\n");
 
     Con *con;
-    TAILQ_FOREACH(con, &all_cons, all_cons) {
+    TAILQ_FOREACH (con, &all_cons, all_cons) {
         if (con->mark && strcmp(con->mark, mark) == 0)
             FREE(con->mark);
     }
@@ -1032,7 +1038,7 @@ void cmd_mark(I3_CMD, char *mark) {
 
     HANDLE_EMPTY_MATCH;
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         current->con->mark = sstrdup(mark);
     }
@@ -1047,19 +1053,19 @@ void cmd_mark(I3_CMD, char *mark) {
  *
  */
 void cmd_unmark(I3_CMD, char *mark) {
-   if (mark == NULL) {
-       Con *con;
-       TAILQ_FOREACH(con, &all_cons, all_cons) {
-           FREE(con->mark);
-       }
-       DLOG("removed all window marks");
-   } else {
-       Con *con;
-       TAILQ_FOREACH(con, &all_cons, all_cons) {
-           if (con->mark && strcmp(con->mark, mark) == 0)
-               FREE(con->mark);
-       }
-       DLOG("removed window mark %s\n", mark);
+    if (mark == NULL) {
+        Con *con;
+        TAILQ_FOREACH (con, &all_cons, all_cons) {
+            FREE(con->mark);
+        }
+        DLOG("removed all window marks");
+    } else {
+        Con *con;
+        TAILQ_FOREACH (con, &all_cons, all_cons) {
+            if (con->mark && strcmp(con->mark, mark) == 0)
+                FREE(con->mark);
+        }
+        DLOG("removed window mark %s\n", mark);
     }
 
     cmd_output->needs_tree_render = true;
@@ -1095,7 +1101,7 @@ void cmd_move_con_to_output(I3_CMD, char *name) {
     Output *output;
 
     // TODO: fix the handling of criteria
-    TAILQ_FOREACH(current, &owindows, owindows)
+    TAILQ_FOREACH (current, &owindows, owindows)
         current_output = get_output_of_con(current->con);
 
     assert(current_output != NULL);
@@ -1126,7 +1132,7 @@ void cmd_move_con_to_output(I3_CMD, char *name) {
         return;
     }
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         con_move_to_workspace(current->con, ws, true, false);
     }
@@ -1147,7 +1153,7 @@ void cmd_floating(I3_CMD, char *floating_mode) {
 
     HANDLE_EMPTY_MATCH;
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         if (strcmp(floating_mode, "toggle") == 0) {
             DLOG("should toggle mode\n");
@@ -1177,7 +1183,7 @@ void cmd_move_workspace_to_output(I3_CMD, char *name) {
     HANDLE_EMPTY_MATCH;
 
     owindow *current;
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         Output *current_output = get_output_of_con(current->con);
         if (!current_output) {
             ELOG("Cannot get current output. This is a bug in i3.\n");
@@ -1207,13 +1213,13 @@ void cmd_move_workspace_to_output(I3_CMD, char *name) {
             /* check if we can find a workspace assigned to this output */
             bool used_assignment = false;
             struct Workspace_Assignment *assignment;
-            TAILQ_FOREACH(assignment, &ws_assignments, ws_assignments) {
+            TAILQ_FOREACH (assignment, &ws_assignments, ws_assignments) {
                 if (strcmp(assignment->output, current_output->name) != 0)
                     continue;
 
                 /* check if this workspace is already attached to the tree */
                 Con *workspace = NULL, *out;
-                TAILQ_FOREACH(out, &(croot->nodes_head), nodes)
+                TAILQ_FOREACH (out, &(croot->nodes_head), nodes)
                     GREP_FIRST(workspace, output_get_content(out),
                                !strcasecmp(child->name, assignment->name));
                 if (workspace != NULL)
@@ -1250,7 +1256,7 @@ void cmd_move_workspace_to_output(I3_CMD, char *name) {
 
         /* fix the coordinates of the floating containers */
         Con *floating_con;
-        TAILQ_FOREACH(floating_con, &(ws->floating_head), floating_windows)
+        TAILQ_FOREACH (floating_con, &(ws->floating_head), floating_windows)
             floating_fix_coordinates(floating_con, &(old_content->rect), &(content->rect));
 
         ipc_send_event("workspace", I3_IPC_EVENT_WORKSPACE, "{\"change\":\"move\"}");
@@ -1264,7 +1270,7 @@ void cmd_move_workspace_to_output(I3_CMD, char *name) {
          * focus order/number of other workspaces on the output.
          * Instead, we loop through the available workspaces and only work with
          * previously_visible_ws if we still find it. */
-        TAILQ_FOREACH(ws, &(content->nodes_head), nodes) {
+        TAILQ_FOREACH (ws, &(content->nodes_head), nodes) {
             if (ws != previously_visible_ws)
                 continue;
 
@@ -1292,7 +1298,7 @@ void cmd_split(I3_CMD, char *direction) {
     if (match_is_empty(current_match))
         tree_split(focused, (direction[0] == 'v' ? VERT : HORIZ));
     else {
-        TAILQ_FOREACH(current, &owindows, owindows) {
+        TAILQ_FOREACH (current, &owindows, owindows) {
             DLOG("matching: %p / %s\n", current->con, current->con->name);
             tree_split(current->con, (direction[0] == 'v' ? VERT : HORIZ));
         }
@@ -1329,7 +1335,7 @@ void cmd_kill(I3_CMD, char *kill_mode_str) {
     if (match_is_empty(current_match))
         tree_close_con(kill_mode);
     else {
-        TAILQ_FOREACH(current, &owindows, owindows) {
+        TAILQ_FOREACH (current, &owindows, owindows) {
             DLOG("matching: %p / %s\n", current->con, current->con->name);
             tree_close(current->con, kill_mode, false, false);
         }
@@ -1394,9 +1400,10 @@ void cmd_focus_window_mode(I3_CMD, char *window_mode) {
             current = TAILQ_FIRST(&(ws->focus_head));
             if (current != NULL && current->type == CT_FLOATING_CON)
                 window_mode = "tiling";
-            else window_mode = "floating";
+            else
+                window_mode = "floating";
         }
-        TAILQ_FOREACH(current, &(ws->focus_head), focused) {
+        TAILQ_FOREACH (current, &(ws->focus_head), focused) {
             if ((strcmp(window_mode, "floating") == 0 && current->type != CT_FLOATING_CON) ||
                 (strcmp(window_mode, "tiling") == 0 && current->type == CT_FLOATING_CON))
                 continue;
@@ -1431,7 +1438,8 @@ void cmd_focus_level(I3_CMD, char *level) {
     }
 
     /* Focusing a child should always be allowed. */
-    else success = level_down();
+    else
+        success = level_down();
 
     cmd_output->needs_tree_render = success;
     // XXX: default reply for now, make this a better reply
@@ -1457,7 +1465,7 @@ void cmd_focus(I3_CMD) {
     Con *__i3_scratch = workspace_get("__i3_scratch", NULL);
     int count = 0;
     owindow *current;
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         Con *ws = con_get_workspace(current->con);
         /* If no workspace could be found, this was a dock window.
          * Just skip it, you cannot focus dock windows. */
@@ -1506,7 +1514,8 @@ void cmd_focus(I3_CMD) {
 
     if (count > 1)
         LOG("WARNING: Your criteria for the focus command matches %d containers, "
-            "while only exactly one container can be focused at a time.\n", count);
+            "while only exactly one container can be focused at a time.\n",
+            count);
 
     cmd_output->needs_tree_render = true;
     // XXX: default reply for now, make this a better reply
@@ -1525,7 +1534,7 @@ void cmd_fullscreen(I3_CMD, char *fullscreen_mode) {
 
     HANDLE_EMPTY_MATCH;
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         con_toggle_fullscreen(current->con, (strcmp(fullscreen_mode, "global") == 0 ? CF_GLOBAL : CF_OUTPUT));
     }
@@ -1559,10 +1568,7 @@ void cmd_move_direction(I3_CMD, char *direction, char *move_px) {
         }
         floating_reposition(focused->parent, newrect);
     } else {
-        tree_move((strcmp(direction, "right") == 0 ? D_RIGHT :
-                   (strcmp(direction, "left") == 0 ? D_LEFT :
-                    (strcmp(direction, "up") == 0 ? D_UP :
-                     D_DOWN))));
+        tree_move((strcmp(direction, "right") == 0 ? D_RIGHT : (strcmp(direction, "left") == 0 ? D_LEFT : (strcmp(direction, "up") == 0 ? D_UP : D_DOWN))));
         cmd_output->needs_tree_render = true;
     }
 
@@ -1601,7 +1607,7 @@ void cmd_layout(I3_CMD, char *layout_str) {
     if (match_is_empty(current_match))
         con_set_layout(focused, layout);
     else {
-        TAILQ_FOREACH(current, &owindows, owindows) {
+        TAILQ_FOREACH (current, &owindows, owindows) {
             DLOG("matching: %p / %s\n", current->con, current->con->name);
             con_set_layout(current->con, layout);
         }
@@ -1628,7 +1634,7 @@ void cmd_layout_toggle(I3_CMD, char *toggle_mode) {
     if (match_is_empty(current_match))
         con_toggle_layout(focused, toggle_mode);
     else {
-        TAILQ_FOREACH(current, &owindows, owindows) {
+        TAILQ_FOREACH (current, &owindows, owindows) {
             DLOG("matching: %p / %s\n", current->con, current->con->name);
             con_toggle_layout(current->con, toggle_mode);
         }
@@ -1645,6 +1651,8 @@ void cmd_layout_toggle(I3_CMD, char *toggle_mode) {
  */
 void cmd_exit(I3_CMD) {
     LOG("Exiting due to user command.\n");
+    ipc_shutdown();
+    unlink(config.ipc_socket_path);
     xcb_disconnect(conn);
     exit(0);
 
@@ -1676,6 +1684,14 @@ void cmd_reload(I3_CMD) {
  */
 void cmd_restart(I3_CMD) {
     LOG("restarting i3\n");
+    ipc_shutdown();
+    /* We need to call this manually since atexit handlers don’t get called
+     * when exec()ing */
+    purge_zerobyte_logfile();
+    /* The unlink call is intentionally after the purge_zerobyte_logfile() so
+     * that the latter does not remove the directory yet. We need to store the
+     * restart layout state in there. */
+    unlink(config.ipc_socket_path);
     i3_restart(false);
 
     // XXX: default reply for now, make this a better reply
@@ -1717,7 +1733,7 @@ void cmd_focus_output(I3_CMD, char *name) {
     Output *current_output = NULL;
     Output *output;
 
-    TAILQ_FOREACH(current, &owindows, owindows)
+    TAILQ_FOREACH (current, &owindows, owindows)
         current_output = get_output_of_con(current->con);
     assert(current_output != NULL);
 
@@ -1749,7 +1765,6 @@ void cmd_focus_output(I3_CMD, char *name) {
  *
  */
 void cmd_move_window_to_position(I3_CMD, char *method, char *cx, char *cy) {
-
     int x = atoi(cx);
     int y = atoi(cy);
 
@@ -1787,7 +1802,6 @@ void cmd_move_window_to_position(I3_CMD, char *method, char *cx, char *cy) {
  *
  */
 void cmd_move_window_to_center(I3_CMD, char *method) {
-
     if (!con_is_floating(focused)) {
         ELOG("Cannot change position. The window/container is not floating\n");
         yerror("Cannot change position. The window/container is not floating.");
@@ -1798,8 +1812,8 @@ void cmd_move_window_to_center(I3_CMD, char *method) {
         Rect *rect = &focused->parent->rect;
 
         DLOG("moving to absolute center\n");
-        rect->x = croot->rect.width/2 - rect->width/2;
-        rect->y = croot->rect.height/2 - rect->height/2;
+        rect->x = croot->rect.width / 2 - rect->width / 2;
+        rect->y = croot->rect.height / 2 - rect->height / 2;
 
         floating_maybe_reassign_ws(focused->parent);
         cmd_output->needs_tree_render = true;
@@ -1810,8 +1824,8 @@ void cmd_move_window_to_center(I3_CMD, char *method) {
         Rect newrect = focused->parent->rect;
 
         DLOG("moving to center\n");
-        newrect.x = wsrect->width/2 - newrect.width/2;
-        newrect.y = wsrect->height/2 - newrect.height/2;
+        newrect.x = wsrect->width / 2 - newrect.width / 2;
+        newrect.y = wsrect->height / 2 - newrect.height / 2;
 
         floating_reposition(focused->parent, newrect);
     }
@@ -1830,7 +1844,7 @@ void cmd_move_scratchpad(I3_CMD) {
 
     HANDLE_EMPTY_MATCH;
 
-    TAILQ_FOREACH(current, &owindows, owindows) {
+    TAILQ_FOREACH (current, &owindows, owindows) {
         DLOG("matching: %p / %s\n", current->con, current->con->name);
         scratchpad_move(current->con);
     }
@@ -1851,7 +1865,7 @@ void cmd_scratchpad_show(I3_CMD) {
     if (match_is_empty(current_match)) {
         scratchpad_show(NULL);
     } else {
-        TAILQ_FOREACH(current, &owindows, owindows) {
+        TAILQ_FOREACH (current, &owindows, owindows) {
             DLOG("matching: %p / %s\n", current->con, current->con->name);
             scratchpad_show(current->con);
         }
@@ -1867,6 +1881,11 @@ void cmd_scratchpad_show(I3_CMD) {
  *
  */
 void cmd_rename_workspace(I3_CMD, char *old_name, char *new_name) {
+    if (strncasecmp(new_name, "__", strlen("__")) == 0) {
+        LOG("Cannot rename workspace to \"%s\": names starting with __ are i3-internal.", new_name);
+        ysuccess(false);
+        return;
+    }
     if (old_name) {
         LOG("Renaming workspace \"%s\" to \"%s\"\n", old_name, new_name);
     } else {
@@ -1875,9 +1894,9 @@ void cmd_rename_workspace(I3_CMD, char *old_name, char *new_name) {
 
     Con *output, *workspace = NULL;
     if (old_name) {
-        TAILQ_FOREACH(output, &(croot->nodes_head), nodes)
+        TAILQ_FOREACH (output, &(croot->nodes_head), nodes)
             GREP_FIRST(workspace, output_get_content(output),
-                !strcasecmp(child->name, old_name));
+                       !strcasecmp(child->name, old_name));
     } else {
         workspace = con_get_workspace(focused);
     }
@@ -1891,9 +1910,9 @@ void cmd_rename_workspace(I3_CMD, char *old_name, char *new_name) {
     }
 
     Con *check_dest = NULL;
-    TAILQ_FOREACH(output, &(croot->nodes_head), nodes)
+    TAILQ_FOREACH (output, &(croot->nodes_head), nodes)
         GREP_FIRST(check_dest, output_get_content(output),
-            !strcasecmp(child->name, new_name));
+                   !strcasecmp(child->name, new_name));
 
     if (check_dest != NULL) {
         // TODO: we should include the new workspace name here and use yajl for
@@ -1913,7 +1932,8 @@ void cmd_rename_workspace(I3_CMD, char *old_name, char *new_name) {
         parsed_num < 0 ||
         endptr == new_name)
         workspace->num = -1;
-    else workspace->num = parsed_num;
+    else
+        workspace->num = parsed_num;
     LOG("num = %d\n", workspace->num);
 
     /* By re-attaching, the sort order will be correct afterwards. */
@@ -1952,7 +1972,7 @@ bool cmd_bar_mode(char *bar_mode, char *bar_id) {
 
     bool changed_sth = false;
     Barconfig *current = NULL;
-    TAILQ_FOREACH(current, &barconfigs, configs) {
+    TAILQ_FOREACH (current, &barconfigs, configs) {
         if (bar_id && strcmp(current->id, bar_id) != 0)
             continue;
 
@@ -1964,7 +1984,7 @@ bool cmd_bar_mode(char *bar_mode, char *bar_id) {
         changed_sth = true;
 
         if (bar_id)
-             break;
+            break;
     }
 
     if (bar_id && !changed_sth) {
@@ -1995,7 +2015,7 @@ bool cmd_bar_hidden_state(char *bar_hidden_state, char *bar_id) {
 
     bool changed_sth = false;
     Barconfig *current = NULL;
-    TAILQ_FOREACH(current, &barconfigs, configs) {
+    TAILQ_FOREACH (current, &barconfigs, configs) {
         if (bar_id && strcmp(current->id, bar_id) != 0)
             continue;
 
@@ -2007,7 +2027,7 @@ bool cmd_bar_hidden_state(char *bar_hidden_state, char *bar_id) {
         changed_sth = true;
 
         if (bar_id)
-             break;
+            break;
     }
 
     if (bar_id && !changed_sth) {
@@ -2045,7 +2065,7 @@ void cmd_bar(I3_CMD, char *bar_type, char *bar_value, char *bar_id) {
  *
  */
 void cmd_shmlog(I3_CMD, char *argument) {
-    if (!strcmp(argument,"toggle"))
+    if (!strcmp(argument, "toggle"))
         /* Toggle shm log, if size is not 0. If it is 0, set it to default. */
         shmlog_size = shmlog_size ? -shmlog_size : default_shmlog_size;
     else if (!strcmp(argument, "on"))
@@ -2077,7 +2097,7 @@ void cmd_shmlog(I3_CMD, char *argument) {
  */
 void cmd_debuglog(I3_CMD, char *argument) {
     bool logging = get_debug_logging();
-    if (!strcmp(argument,"toggle")) {
+    if (!strcmp(argument, "toggle")) {
         LOG("%s debug logging\n", logging ? "Disabling" : "Enabling");
         set_debug_logging(!logging);
     } else if (!strcmp(argument, "on") && !logging) {