aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--sway/commands.c175
-rw-r--r--sway/handlers.c49
-rw-r--r--sway/handlers.h17
-rw-r--r--sway/main.c25
-rw-r--r--sway/movement.c10
-rw-r--r--sway/movement.h2
6 files changed, 158 insertions, 120 deletions
diff --git a/sway/commands.c b/sway/commands.c
index 870a2377..7721c6fb 100644
--- a/sway/commands.c
+++ b/sway/commands.c
@@ -5,6 +5,8 @@
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
+#include <sys/wait.h>
+#include <fcntl.h>
#include <ctype.h>
#include "stringop.h"
#include "layout.h"
@@ -29,11 +31,44 @@ struct modifier_key modifiers[] = {
{ "Mod5", WLC_BIT_MOD_MOD5 },
};
-bool cmd_bindsym(struct sway_config *config, int argc, char **argv) {
- if (argc < 2) {
- sway_log(L_ERROR, "Invalid set command (expected 2 arguments, got %d)", argc);
- return false;
+enum expected_args {
+ EXPECTED_MORE_THEN,
+ EXPECTED_LESS_THEN,
+ EXPECTED_SAME_AS
+};
+
+static bool checkarg(int argc, char *name, enum expected_args type, int val) {
+ switch(type) {
+ case EXPECTED_MORE_THEN:
+ if (argc > val) {
+ return true;
+ }
+ sway_log(L_ERROR, "Invalid %s command."
+ "(expected more then %d arguments, got %d", name, val, argc);
+ break;
+ case EXPECTED_LESS_THEN:
+ if (argc < val) {
+ return true;
+ };
+ sway_log(L_ERROR, "Invalid %s command."
+ "(expected less then %d arguments, got %d", name, val, argc);
+ break;
+ case EXPECTED_SAME_AS:
+ if (argc == val) {
+ return true;
+ };
+ sway_log(L_ERROR, "Invalid %s command."
+ "(expected %d arguments, got %d", name, val, argc);
+ break;
}
+ return false;
+}
+
+
+static bool cmd_bindsym(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "bindsym", EXPECTED_MORE_THEN, 1) == false) {
+ return false;
+ };
struct sway_binding *binding = malloc(sizeof(struct sway_binding));
binding->keys = create_list();
@@ -73,46 +108,71 @@ bool cmd_bindsym(struct sway_config *config, int argc, char **argv) {
return true;
}
-bool cmd_exec(struct sway_config *config, int argc, char **argv) {
- if (argc < 1) {
- sway_log(L_ERROR, "Invalid exec command (expected at least 1 argument, got %d)", argc);
- return false;
+static void cmd_exec_cleanup(int signal) {
+ while (waitpid((pid_t)-1, 0, WNOHANG) > 0){};
+}
+
+static bool cmd_exec_always(struct sway_config *config, int argc, char **argv) {
+ /* setup signal handler to cleanup dead proccesses */
+ /* TODO: replace this with a function that has constructor attribute? */
+ static bool cleanup = false;
+ if(cleanup == false) {
+ signal(SIGCHLD, cmd_exec_cleanup);
+ cleanup = true;
+ /* Set it so filedescriptors are closed for executed programs */
+ int flag_out = fcntl(STDOUT_FILENO, F_GETFD);
+ int flag_in = fcntl(STDIN_FILENO, F_GETFD);
+ int flag_err = fcntl(STDERR_FILENO, F_GETFD);
+ if (flag_out != -1) {
+ flag_out |= FD_CLOEXEC;
+ fcntl(STDOUT_FILENO, F_SETFD, flag_out);
+ }
+ if (flag_in != -1) {
+ flag_in |= FD_CLOEXEC;
+ fcntl(STDIN_FILENO, F_SETFD, flag_in);
+ }
+ if (flag_err != -1) {
+ flag_err |= FD_CLOEXEC;
+ fcntl(STDERR_FILENO, F_SETFD, flag_err);
+ }
}
- if (config->reloading) {
- sway_log(L_DEBUG, "Ignoring exec %s due to reload", join_args(argv, argc));
- return true;
+ if (checkarg(argc, "exec_always", EXPECTED_MORE_THEN, 0) == false) {
+ return false;
}
- if (fork() == 0) {
+ pid_t pid = fork();
+ /* Failed to fork */
+ if (pid < 0) {
+ sway_log(L_ERROR, "exec command failed, sway did not fork");
+ return false;
+ }
+ /* Child process */
+ if (pid == 0) {
char *args = join_args(argv, argc);
sway_log(L_DEBUG, "Executing %s", args);
execl("/bin/sh", "sh", "-c", args, (char *)NULL);
+ /* Execl doesnt return unless failure */
+ sway_log(L_ERROR, "could not find /bin/sh");
free(args);
- exit(0);
+ exit(-1);
}
+ /* Parent */
return true;
}
-bool cmd_exec_always(struct sway_config *config, int argc, char **argv) {
- if (argc < 1) {
- sway_log(L_ERROR, "Invalid exec_always command (expected at least 1 argument, got %d)", argc);
- return false;
- }
-
- if (fork() == 0) {
+static bool cmd_exec(struct sway_config *config, int argc, char **argv) {
+ if (config->reloading) {
char *args = join_args(argv, argc);
- sway_log(L_DEBUG, "Executing %s", args);
- execl("/bin/sh", "sh", "-c", args, (char *)NULL);
+ sway_log(L_DEBUG, "Ignoring exec %s due to reload", args);
free(args);
- exit(0);
+ return true;
}
- return true;
+ return cmd_exec_always(config, argc, argv);
}
-bool cmd_exit(struct sway_config *config, int argc, char **argv) {
- if (argc != 0) {
- sway_log(L_ERROR, "Invalid exit command (expected 1 arguments, got %d)", argc);
+static bool cmd_exit(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "exit", EXPECTED_SAME_AS, 0) == false) {
return false;
}
// TODO: Some kind of clean up is probably in order
@@ -120,9 +180,8 @@ bool cmd_exit(struct sway_config *config, int argc, char **argv) {
return true;
}
-bool cmd_focus(struct sway_config *config, int argc, char **argv) {
- if (argc != 1) {
- sway_log(L_ERROR, "Invalid focus command (expected 1 arguments, got %d)", argc);
+static bool cmd_focus(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "focus", EXPECTED_SAME_AS, 1) == false) {
return false;
}
if (strcasecmp(argv[0], "left") == 0) {
@@ -139,9 +198,8 @@ bool cmd_focus(struct sway_config *config, int argc, char **argv) {
return true;
}
-bool cmd_focus_follows_mouse(struct sway_config *config, int argc, char **argv) {
- if (argc != 1) {
- sway_log(L_ERROR, "Invalid focus_follows_mouse command (expected 1 arguments, got %d)", argc);
+static bool cmd_focus_follows_mouse(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "focus_follows_mouse", EXPECTED_SAME_AS, 1) == false) {
return false;
}
@@ -149,9 +207,8 @@ bool cmd_focus_follows_mouse(struct sway_config *config, int argc, char **argv)
return true;
}
-bool cmd_layout(struct sway_config *config, int argc, char **argv) {
- if (argc < 1) {
- sway_log(L_ERROR, "Invalid layout command (expected at least 1 argument, got %d)", argc);
+static bool cmd_layout(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "layout", EXPECTED_MORE_THEN, 0) == false) {
return false;
}
swayc_t *parent = get_focused_container(&root_container);
@@ -174,9 +231,8 @@ bool cmd_layout(struct sway_config *config, int argc, char **argv) {
return true;
}
-bool cmd_reload(struct sway_config *config, int argc, char **argv) {
- if (argc != 0) {
- sway_log(L_ERROR, "Invalid reload command (expected 0 arguments, got %d)", argc);
+static bool cmd_reload(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "reload", EXPECTED_SAME_AS, 0) == false) {
return false;
}
if (!load_config()) {
@@ -186,9 +242,8 @@ bool cmd_reload(struct sway_config *config, int argc, char **argv) {
return true;
}
-bool cmd_set(struct sway_config *config, int argc, char **argv) {
- if (argc != 2) {
- sway_log(L_ERROR, "Invalid set command (expected 2 arguments, got %d)", argc);
+static bool cmd_set(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "set", EXPECTED_SAME_AS, 2) == false) {
return false;
}
struct sway_variable *var = malloc(sizeof(struct sway_variable));
@@ -200,9 +255,11 @@ bool cmd_set(struct sway_config *config, int argc, char **argv) {
return true;
}
-bool _do_split(struct sway_config *config, int argc, char **argv, int layout) {
- if (argc != 0) {
- sway_log(L_ERROR, "Invalid splitv command (expected 0 arguments, got %d)", argc);
+static bool _do_split(struct sway_config *config, int argc, char **argv, int layout) {
+ char *name = layout == L_VERT ? "splitv":
+ layout == L_HORIZ ? "splith":
+ "split";
+ if (checkarg(argc, name, EXPECTED_SAME_AS, 0) == false) {
return false;
}
swayc_t *focused = get_focused_container(&root_container);
@@ -225,20 +282,18 @@ bool _do_split(struct sway_config *config, int argc, char **argv, int layout) {
return true;
}
-bool cmd_splitv(struct sway_config *config, int argc, char **argv) {
+static bool cmd_splitv(struct sway_config *config, int argc, char **argv) {
return _do_split(config, argc, argv, L_VERT);
}
-bool cmd_splith(struct sway_config *config, int argc, char **argv) {
+static bool cmd_splith(struct sway_config *config, int argc, char **argv) {
return _do_split(config, argc, argv, L_HORIZ);
}
-bool cmd_log_colors(struct sway_config *config, int argc, char **argv) {
- if (argc != 1) {
- sway_log(L_ERROR, "Invalid log_colors command (expected 1 argument, got %d)", argc);
+static bool cmd_log_colors(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "log_colors", EXPECTED_SAME_AS, 1) == false) {
return false;
}
-
if (strcasecmp(argv[0], "no") != 0 && strcasecmp(argv[0], "yes") != 0) {
sway_log(L_ERROR, "Invalid log_colors command (expected `yes` or `no`, got '%s')", argv[0]);
return false;
@@ -248,9 +303,8 @@ bool cmd_log_colors(struct sway_config *config, int argc, char **argv) {
return true;
}
-bool cmd_fullscreen(struct sway_config *config, int argc, char **argv) {
- if (argc != 1) {
- sway_log(L_ERROR, "Invalid fullscreen command (expected 1 arguments, got %d)", argc);
+static bool cmd_fullscreen(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "fullscreen", EXPECTED_SAME_AS, 0) == false) {
return false;
}
@@ -262,9 +316,8 @@ bool cmd_fullscreen(struct sway_config *config, int argc, char **argv) {
return true;
}
-bool cmd_workspace(struct sway_config *config, int argc, char **argv) {
- if (argc != 1) {
- sway_log(L_ERROR, "Invalid workspace command (expected 1 arguments, got %d)", argc);
+static bool cmd_workspace(struct sway_config *config, int argc, char **argv) {
+ if (checkarg(argc, "workspace", EXPECTED_SAME_AS, 1) == false) {
return false;
}
@@ -278,7 +331,7 @@ bool cmd_workspace(struct sway_config *config, int argc, char **argv) {
}
/* Keep alphabetized */
-struct cmd_handler handlers[] = {
+static struct cmd_handler handlers[] = {
{ "bindsym", cmd_bindsym },
{ "exec", cmd_exec },
{ "exec_always", cmd_exec_always },
@@ -295,7 +348,7 @@ struct cmd_handler handlers[] = {
{ "workspace", cmd_workspace }
};
-char **split_directive(char *line, int *argc) {
+static char **split_directive(char *line, int *argc) {
const char *delimiters = " ";
*argc = 0;
while (isspace(*line) && *line) ++line;
@@ -347,13 +400,13 @@ char **split_directive(char *line, int *argc) {
return parts;
}
-int handler_compare(const void *_a, const void *_b) {
+static int handler_compare(const void *_a, const void *_b) {
const struct cmd_handler *a = _a;
const struct cmd_handler *b = _b;
return strcasecmp(a->command, b->command);
}
-struct cmd_handler *find_handler(struct cmd_handler handlers[], int l, char *line) {
+static struct cmd_handler *find_handler(struct cmd_handler handlers[], int l, char *line) {
struct cmd_handler d = { .command=line };
struct cmd_handler *res = bsearch(&d, handlers, l, sizeof(struct cmd_handler), handler_compare);
return res;
diff --git a/sway/handlers.c b/sway/handlers.c
index 78ca1363..fe7de75b 100644
--- a/sway/handlers.c
+++ b/sway/handlers.c
@@ -9,16 +9,16 @@
#include "commands.h"
#include "handlers.h"
-bool handle_output_created(wlc_handle output) {
+static bool handle_output_created(wlc_handle output) {
add_output(output);
return true;
}
-void handle_output_destroyed(wlc_handle output) {
+static void handle_output_destroyed(wlc_handle output) {
destroy_output(output);
}
-void handle_output_resolution_change(wlc_handle output, const struct wlc_size *from, const struct wlc_size *to) {
+static void handle_output_resolution_change(wlc_handle output, const struct wlc_size *from, const struct wlc_size *to) {
sway_log(L_DEBUG, "Output %d resolution changed to %d x %d", output, to->w, to->h);
swayc_t *c = get_swayc_for_handle(output, &root_container);
if (!c) return;
@@ -27,7 +27,7 @@ void handle_output_resolution_change(wlc_handle output, const struct wlc_size *f
arrange_windows(&root_container, -1, -1);
}
-void handle_output_focused(wlc_handle output, bool focus) {
+static void handle_output_focused(wlc_handle output, bool focus) {
swayc_t *c = get_swayc_for_handle(output, &root_container);
if (!c) return;
if (focus) {
@@ -36,27 +36,26 @@ void handle_output_focused(wlc_handle output, bool focus) {
}
}
-bool handle_view_created(wlc_handle view) {
+static bool handle_view_created(wlc_handle view) {
add_view(view);
return true;
}
-void handle_view_destroyed(wlc_handle view) {
+static void handle_view_destroyed(wlc_handle view) {
sway_log(L_DEBUG, "Destroying window %d", view);
destroy_view(get_swayc_for_handle(view, &root_container));
- return true;
}
-void handle_view_focus(wlc_handle view, bool focus) {
+static void handle_view_focus(wlc_handle view, bool focus) {
return;
}
-void handle_view_geometry_request(wlc_handle view, const struct wlc_geometry* geometry) {
+static void handle_view_geometry_request(wlc_handle view, const struct wlc_geometry* geometry) {
// deny that shit
}
-bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers
+static bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers
*modifiers, uint32_t key, uint32_t sym, enum wlc_key_state state) {
enum { QSIZE = 32 };
static uint8_t head = 0;
@@ -133,7 +132,7 @@ bool pointer_test(swayc_t *view, void *_origin) {
struct wlc_origin mouse_origin;
-bool handle_pointer_motion(wlc_handle view, uint32_t time, const struct wlc_origin *origin) {
+static bool handle_pointer_motion(wlc_handle view, uint32_t time, const struct wlc_origin *origin) {
mouse_origin = *origin;
if (!config->focus_follows_mouse) {
return true;
@@ -148,7 +147,7 @@ bool handle_pointer_motion(wlc_handle view, uint32_t time, const struct wlc_orig
return true;
}
-bool handle_pointer_button(wlc_handle view, uint32_t time, const struct wlc_modifiers *modifiers,
+static bool handle_pointer_button(wlc_handle view, uint32_t time, const struct wlc_modifiers *modifiers,
uint32_t button, enum wlc_button_state state) {
if (state == WLC_BUTTON_STATE_PRESSED) {
swayc_t *c = find_container(&root_container, pointer_test, &mouse_origin);
@@ -163,3 +162,29 @@ bool handle_pointer_button(wlc_handle view, uint32_t time, const struct wlc_modi
}
return true;
}
+
+
+struct wlc_interface interface = {
+ .output = {
+ .created = handle_output_created,
+ .destroyed = handle_output_destroyed,
+ .resolution = handle_output_resolution_change,
+ .focus = handle_output_focused
+ },
+ .view = {
+ .created = handle_view_created,
+ .destroyed = handle_view_destroyed,
+ .focus = handle_view_focus,
+ .request = {
+ .geometry = handle_view_geometry_request
+ }
+ },
+ .keyboard = {
+ .key = handle_key
+ },
+ .pointer = {
+ .motion = handle_pointer_motion,
+ .button = handle_pointer_button
+ }
+};
+
diff --git a/sway/handlers.h b/sway/handlers.h
index 9792b6d7..798b3b50 100644
--- a/sway/handlers.h
+++ b/sway/handlers.h
@@ -4,21 +4,6 @@
#include <stdbool.h>
#include <wlc/wlc.h>
-bool handle_output_created(wlc_handle output);
-void handle_output_destroyed(wlc_handle output);
-void handle_output_resolution_change(wlc_handle output, const struct wlc_size *from, const struct wlc_size *to);
-void handle_output_focused(wlc_handle output, bool focus);
-
-bool handle_view_created(wlc_handle view);
-void handle_view_destroyed(wlc_handle view);
-void handle_view_focus(wlc_handle view, bool focus);
-void handle_view_geometry_request(wlc_handle view, const struct wlc_geometry* geometry);
-
-bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers
- *modifiers, uint32_t key, uint32_t sym, enum wlc_key_state state);
-
-bool handle_pointer_motion(wlc_handle view, uint32_t time, const struct wlc_origin *origin);
-bool handle_pointer_button(wlc_handle view, uint32_t time, const struct wlc_modifiers *modifiers,
- uint32_t button, enum wlc_button_state state);
+extern struct wlc_interface interface;
#endif
diff --git a/sway/main.c b/sway/main.c
index a7814364..7661551d 100644
--- a/sway/main.c
+++ b/sway/main.c
@@ -12,31 +12,6 @@ int main(int argc, char **argv) {
init_log(L_DEBUG); // TODO: Control this with command line arg
init_layout();
- static struct wlc_interface interface = {
- .output = {
- .created = handle_output_created,
- .destroyed = handle_output_destroyed,
- .resolution = handle_output_resolution_change,
- .focus = handle_output_focused
- },
- .view = {
- .created = handle_view_created,
- .destroyed = handle_view_destroyed,
- .focus = handle_view_focus,
- .request = {
- .geometry = handle_view_geometry_request
- }
- },
- .keyboard = {
- .key = handle_key
- },
- .pointer = {
- .motion = handle_pointer_motion,
- .button = handle_pointer_button
- }
-
- };
-
setenv("WLC_DIM", "0", 0);
if (!wlc_init(&interface, argc, argv)) {
return 1;
diff --git a/sway/movement.c b/sway/movement.c
index a55d0350..166e6508 100644
--- a/sway/movement.c
+++ b/sway/movement.c
@@ -5,7 +5,7 @@
#include "layout.h"
#include "movement.h"
-int move_focus(enum movement_direction direction) {
+bool move_focus(enum movement_direction direction) {
swayc_t *current = get_focused_container(&root_container);
swayc_t *parent = current->parent;
@@ -14,12 +14,12 @@ int move_focus(enum movement_direction direction) {
parent = parent->parent;
if (parent->type == C_ROOT) {
sway_log(L_DEBUG, "Focus cannot move to parent");
- return 1;
+ return false;
} else {
sway_log(L_DEBUG, "Moving focus away from %p", current);
unfocus_all(parent);
focus_view(parent);
- return 0;
+ return true;
}
}
@@ -56,7 +56,7 @@ int move_focus(enum movement_direction direction) {
} else {
unfocus_all(&root_container);
focus_view(parent->children->items[desired]);
- return 0;
+ return true;
}
}
if (!can_move) {
@@ -65,7 +65,7 @@ int move_focus(enum movement_direction direction) {
parent = parent->parent;
if (parent->type == C_ROOT) {
// Nothing we can do
- return 1;
+ return false;
}
}
}
diff --git a/sway/movement.h b/sway/movement.h
index a527674c..dd701877 100644
--- a/sway/movement.h
+++ b/sway/movement.h
@@ -12,6 +12,6 @@ enum movement_direction {
MOVE_PARENT
};
-int move_focus(enum movement_direction direction);
+bool move_focus(enum movement_direction direction);
#endif
generated by cgit v1.2.3 (git 2.39.1) at 2025-04-03 01:46:18 +0000