aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--sway/commands.c100
-rw-r--r--sway/commands.h2
-rw-r--r--sway/handlers.c6
3 files changed, 54 insertions, 54 deletions
diff --git a/sway/commands.c b/sway/commands.c
index f1440c57..41141f87 100644
--- a/sway/commands.c
+++ b/sway/commands.c
@@ -29,10 +29,10 @@ struct modifier_key modifiers[] = {
{ "Mod5", WLC_BIT_MOD_MOD5 },
};
-int cmd_bindsym(struct sway_config *config, int argc, char **argv) {
+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 1;
+ return false;
}
argv[0] = do_var_replacement(config, argv[0]);
@@ -59,7 +59,7 @@ int cmd_bindsym(struct sway_config *config, int argc, char **argv) {
xkb_keysym_t sym = xkb_keysym_from_name(split->items[i], XKB_KEYSYM_CASE_INSENSITIVE);
if (!sym) {
sway_log(L_ERROR, "bindsym - unknown key %s", (char *)split->items[i]);
- return 1;
+ return false;
}
xkb_keysym_t *key = malloc(sizeof(xkb_keysym_t));
*key = sym;
@@ -71,18 +71,18 @@ int cmd_bindsym(struct sway_config *config, int argc, char **argv) {
list_add(config->current_mode->bindings, binding);
sway_log(L_DEBUG, "bindsym - Bound %s to command %s", argv[0], binding->command);
- return 0;
+ return true;
}
-int cmd_exec(struct sway_config *config, int argc, char **argv) {
+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 1;
+ return false;
}
if (config->reloading) {
sway_log(L_DEBUG, "Ignoring exec %s due to reload", join_args(argv, argc));
- return 0;
+ return true;
}
if (fork() == 0) {
@@ -92,13 +92,13 @@ int cmd_exec(struct sway_config *config, int argc, char **argv) {
free(args);
exit(0);
}
- return 0;
+ return true;
}
-int cmd_exec_always(struct sway_config *config, int argc, char **argv) {
+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 1;
+ return false;
}
if (fork() == 0) {
@@ -108,23 +108,23 @@ int cmd_exec_always(struct sway_config *config, int argc, char **argv) {
free(args);
exit(0);
}
- return 0;
+ return true;
}
-int cmd_exit(struct sway_config *config, int argc, char **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);
- return 1;
+ return false;
}
// TODO: Some kind of clean up is probably in order
exit(0);
- return 0;
+ return true;
}
-int cmd_focus(struct sway_config *config, int argc, char **argv) {
+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);
- return 1;
+ return false;
}
if (strcasecmp(argv[0], "left") == 0) {
return move_focus(MOVE_LEFT);
@@ -137,23 +137,23 @@ int cmd_focus(struct sway_config *config, int argc, char **argv) {
} else if (strcasecmp(argv[0], "parent") == 0) {
return move_focus(MOVE_PARENT);
}
- return 0;
+ return true;
}
-int cmd_focus_follows_mouse(struct sway_config *config, int argc, char **argv) {
+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);
- return 1;
+ return false;
}
config->focus_follows_mouse = !strcasecmp(argv[0], "yes");
- return 0;
+ return true;
}
-int cmd_layout(struct sway_config *config, int argc, char **argv) {
+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);
- return 1;
+ return false;
}
swayc_t *parent = get_focused_container(&root_container);
while (parent->type == C_VIEW) {
@@ -172,25 +172,25 @@ int cmd_layout(struct sway_config *config, int argc, char **argv) {
}
arrange_windows(parent, parent->width, parent->height);
- return 0;
+ return true;
}
-int cmd_reload(struct sway_config *config, int argc, char **argv) {
+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);
- return 1;
+ return false;
}
if (!load_config()) {
- return 1;
+ return false;
}
arrange_windows(&root_container, -1, -1);
- return 0;
+ return true;
}
-int cmd_set(struct sway_config *config, int argc, char **argv) {
+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);
- return 1;
+ return false;
}
struct sway_variable *var = malloc(sizeof(struct sway_variable));
var->name = malloc(strlen(argv[0]) + 1);
@@ -198,13 +198,13 @@ int cmd_set(struct sway_config *config, int argc, char **argv) {
var->value = malloc(strlen(argv[1]) + 1);
strcpy(var->value, argv[1]);
list_add(config->symbols, var);
- return 0;
+ return true;
}
-int _do_split(struct sway_config *config, int argc, char **argv, int layout) {
+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);
- return 1;
+ return false;
}
swayc_t *focused = get_focused_container(&root_container);
swayc_t *parent = focused->parent;
@@ -223,36 +223,36 @@ int _do_split(struct sway_config *config, int argc, char **argv, int layout) {
list_add(new_container->children, focused);
focus_view(focused);
arrange_windows(parent, -1, -1);
- return 0;
+ return true;
}
-int cmd_splitv(struct sway_config *config, int argc, char **argv) {
+bool cmd_splitv(struct sway_config *config, int argc, char **argv) {
return _do_split(config, argc, argv, L_VERT);
}
-int cmd_splith(struct sway_config *config, int argc, char **argv) {
+bool cmd_splith(struct sway_config *config, int argc, char **argv) {
return _do_split(config, argc, argv, L_HORIZ);
}
-int cmd_log_colors(struct sway_config *config, int argc, char **argv) {
+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);
- return 1;
+ 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 1;
+ return false;
}
sway_log_colors(!strcasecmp(argv[0], "yes"));
- return 0;
+ return true;
}
-int cmd_fullscreen(struct sway_config *config, int argc, char **argv) {
+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);
- return 1;
+ return false;
}
swayc_t *container = get_focused_container(&root_container);
@@ -260,13 +260,13 @@ int cmd_fullscreen(struct sway_config *config, int argc, char **argv) {
wlc_view_set_state(container->handle, WLC_BIT_FULLSCREEN, !current);
arrange_windows(container, -1, -1);
- return 0;
+ return true;
}
-int cmd_workspace(struct sway_config *config, int argc, char **argv) {
+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);
- return 1;
+ return false;
}
swayc_t *workspace = workspace_find_by_name(argv[0]);
@@ -275,7 +275,7 @@ int cmd_workspace(struct sway_config *config, int argc, char **argv) {
} else sway_log(L_DEBUG, "workspace exists, all ok");
workspace_switch(workspace);
- return 0;
+ return true;
}
/* Keep alphabetized */
@@ -363,7 +363,7 @@ struct cmd_handler *find_handler(struct cmd_handler handlers[], int l, char *lin
int handle_command(struct sway_config *config, char *exec) {
sway_log(L_INFO, "Handling command '%s'", exec);
char *ptr, *cmd;
- int ret;
+ bool exec_success;
if ((ptr = strchr(exec, ' ')) == NULL) {
cmd = exec;
@@ -376,22 +376,22 @@ int handle_command(struct sway_config *config, char *exec) {
struct cmd_handler *handler = find_handler(handlers, sizeof(handlers) / sizeof(struct cmd_handler), cmd);
if (handler == NULL) {
sway_log(L_ERROR, "Unknown command '%s'", cmd);
- ret = 0; // TODO: return error, probably
+ exec_success = false; // TODO: return error, probably
} else {
int argc;
char **argv = split_directive(exec + strlen(handler->command), &argc);
int i;
- ret = handler->handle(config, argc, argv);
+ exec_success = handler->handle(config, argc, argv);
for (i = 0; i < argc; ++i) {
free(argv[i]);
}
free(argv);
- if (ret != 0) {
+ if (!exec_success) {
sway_log(L_ERROR, "Command failed: %s", cmd);
}
}
if(ptr) {
free(cmd);
}
- return ret;
+ return exec_success;
}
diff --git a/sway/commands.h b/sway/commands.h
index 42b39f27..19fd3f33 100644
--- a/sway/commands.h
+++ b/sway/commands.h
@@ -5,7 +5,7 @@
struct cmd_handler {
char *command;
- int (*handle)(struct sway_config *config, int argc, char **argv);
+ bool (*handle)(struct sway_config *config, int argc, char **argv);
};
int handle_command(struct sway_config *config, char *command);
diff --git a/sway/handlers.c b/sway/handlers.c
index cdf76554..938dc3fd 100644
--- a/sway/handlers.c
+++ b/sway/handlers.c
@@ -51,7 +51,7 @@ bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers
// TODO: handle keybindings with more than 1 non-modifier key involved
// Note: reminder to check conflicts with mod+q+a versus mod+q
- bool ret = true;
+ bool cmd_success = true;
struct sway_mode *mode = config->current_mode;
// Lowercase if necessary
@@ -74,12 +74,12 @@ bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers
}
if (match) {
- ret = handle_command(config, binding->command) != 0;
+ cmd_success = handle_command(config, binding->command);
}
}
}
}
- return ret;
+ return cmd_success;
}
bool pointer_test(swayc_t *view, void *_origin) {
generated by cgit v1.2.3 (git 2.39.1) at 2024-11-16 04:54:05 +0000