From e128c7fcd8441f3fa885eafc9328f0937cc7384c Mon Sep 17 00:00:00 2001 From: nicm Date: Mon, 20 May 2019 11:46:06 +0000 Subject: Replace the various identical error callbacks with a single one in cmd-queue.c. --- cmd-command-prompt.c | 19 ++++--------------- cmd-confirm-before.c | 19 ++++--------------- cmd-display-panes.c | 23 +++++++---------------- cmd-queue.c | 19 +++++++++++++++++++ menu.c | 25 ++++++++----------------- server-client.c | 15 ++------------- server.c | 15 ++------------- tmux.h | 1 + 8 files changed, 47 insertions(+), 89 deletions(-) diff --git a/cmd-command-prompt.c b/cmd-command-prompt.c index d7159ad0..a3cc22c8 100644 --- a/cmd-command-prompt.c +++ b/cmd-command-prompt.c @@ -129,17 +129,6 @@ cmd_command_prompt_exec(struct cmd *self, struct cmdq_item *item) return (CMD_RETURN_NORMAL); } -static enum cmd_retval -cmd_command_prompt_error(struct cmdq_item *item, void *data) -{ - char *error = data; - - cmdq_error(item, "%s", error); - free(error); - - return (CMD_RETURN_NORMAL); -} - static int cmd_command_prompt_callback(struct client *c, void *data, const char *s, int done) @@ -177,11 +166,11 @@ cmd_command_prompt_callback(struct client *c, void *data, const char *s, cmdlist = cmd_string_parse(new_template, NULL, 0, &cause); if (cmdlist == NULL) { - if (cause != NULL) { - new_item = cmdq_get_callback(cmd_command_prompt_error, - cause); - } else + if (cause != NULL) + new_item = cmdq_get_error(cause); + else new_item = NULL; + free(cause); } else { new_item = cmdq_get_command(cmdlist, NULL, NULL, 0); cmd_list_free(cmdlist); diff --git a/cmd-confirm-before.c b/cmd-confirm-before.c index 7036d34b..4017a6f9 100644 --- a/cmd-confirm-before.c +++ b/cmd-confirm-before.c @@ -82,17 +82,6 @@ cmd_confirm_before_exec(struct cmd *self, struct cmdq_item *item) return (CMD_RETURN_NORMAL); } -static enum cmd_retval -cmd_confirm_before_error(struct cmdq_item *item, void *data) -{ - char *error = data; - - cmdq_error(item, "%s", error); - free(error); - - return (CMD_RETURN_NORMAL); -} - static int cmd_confirm_before_callback(struct client *c, void *data, const char *s, __unused int done) @@ -112,11 +101,11 @@ cmd_confirm_before_callback(struct client *c, void *data, const char *s, cmdlist = cmd_string_parse(cdata->cmd, NULL, 0, &cause); if (cmdlist == NULL) { - if (cause != NULL) { - new_item = cmdq_get_callback(cmd_confirm_before_error, - cause); - } else + if (cause != NULL) + new_item = cmdq_get_error(cause); + else new_item = NULL; + free(cause); } else { new_item = cmdq_get_command(cmdlist, NULL, NULL, 0); cmd_list_free(cmdlist); diff --git a/cmd-display-panes.c b/cmd-display-panes.c index 8940b429..6e331ae1 100644 --- a/cmd-display-panes.c +++ b/cmd-display-panes.c @@ -182,17 +182,6 @@ cmd_display_panes_draw(struct client *c, struct screen_redraw_ctx *ctx) } } -static enum cmd_retval -cmd_display_panes_error(struct cmdq_item *item, void *data) -{ - char *error = data; - - cmdq_error(item, "%s", error); - free(error); - - return (CMD_RETURN_NORMAL); -} - static void cmd_display_panes_free(struct client *c) { @@ -226,11 +215,13 @@ cmd_display_panes_key(struct client *c, struct key_event *event) cmd = cmd_template_replace(cdata->command, expanded, 1); cmdlist = cmd_string_parse(cmd, NULL, 0, &cause); - if (cmdlist == NULL && cause != NULL) - new_item = cmdq_get_callback(cmd_display_panes_error, cause); - else if (cmdlist == NULL) - new_item = NULL; - else { + if (cmdlist == NULL) { + if (cause != NULL) + new_item = cmdq_get_error(cause); + else + new_item = NULL; + free(cause); + } else { new_item = cmdq_get_command(cmdlist, NULL, NULL, 0); cmd_list_free(cmdlist); } diff --git a/cmd-queue.c b/cmd-queue.c index 93b9788e..68bedae8 100644 --- a/cmd-queue.c +++ b/cmd-queue.c @@ -329,6 +329,25 @@ cmdq_get_callback1(const char *name, cmdq_cb cb, void *data) return (item); } +/* Generic error callback. */ +static enum cmd_retval +cmdq_error_callback(struct cmdq_item *item, void *data) +{ + char *error = data; + + cmdq_error(item, "%s", error); + free(error); + + return (CMD_RETURN_NORMAL); +} + +/* Get an error callback for the command queue. */ +struct cmdq_item * +cmdq_get_error(const char *error) +{ + return (cmdq_get_callback(cmdq_error_callback, xstrdup(error))); +} + /* Fire callback on callback queue. */ static enum cmd_retval cmdq_fire_callback(struct cmdq_item *item) diff --git a/menu.c b/menu.c index 945b62c4..395e1455 100644 --- a/menu.c +++ b/menu.c @@ -191,17 +191,6 @@ menu_free_cb(struct client *c) free(md); } -static enum cmd_retval -menu_error_cb(struct cmdq_item *item, void *data) -{ - char *error = data; - - cmdq_error(item, "%s", error); - free(error); - - return (CMD_RETURN_NORMAL); -} - static int menu_key_cb(struct client *c, struct key_event *event) { @@ -284,12 +273,14 @@ chosen: return (1); } cmdlist = cmd_string_parse(item->command, NULL, 0, &cause); - if (cmdlist == NULL && cause != NULL) - new_item = cmdq_get_callback(menu_error_cb, cause); - else if (cmdlist == NULL) - new_item = NULL; - else { - new_item = cmdq_get_command(cmdlist, &md->fs, NULL, 0); + if (cmdlist == NULL) { + if (cause != NULL) + new_item = cmdq_get_error(cause); + else + new_item = NULL; + free(cause); + } else { + new_item = cmdq_get_command(cmdlist, NULL, NULL, 0); cmd_list_free(cmdlist); } if (new_item != NULL) { diff --git a/server-client.c b/server-client.c index c7a0acc9..e71cce15 100644 --- a/server-client.c +++ b/server-client.c @@ -1768,18 +1768,6 @@ server_client_command_done(struct cmdq_item *item, __unused void *data) return (CMD_RETURN_NORMAL); } -/* Show an error message. */ -static enum cmd_retval -server_client_command_error(struct cmdq_item *item, void *data) -{ - char *error = data; - - cmdq_error(item, "%s", error); - free(error); - - return (CMD_RETURN_NORMAL); -} - /* Handle command message. */ static void server_client_dispatch_command(struct client *c, struct imsg *imsg) @@ -1827,7 +1815,8 @@ server_client_dispatch_command(struct client *c, struct imsg *imsg) return; error: - cmdq_append(c, cmdq_get_callback(server_client_command_error, cause)); + cmdq_append(c, cmdq_get_error(cause)); + free(cause); if (cmdlist != NULL) cmd_list_free(cmdlist); diff --git a/server.c b/server.c index 3c54f453..18818c38 100644 --- a/server.c +++ b/server.c @@ -144,18 +144,6 @@ fail: return (-1); } -/* Server error callback. */ -static enum cmd_retval -server_start_error(struct cmdq_item *item, void *data) -{ - char *error = data; - - cmdq_error(item, "%s", error); - free(error); - - return (CMD_RETURN_NORMAL); -} - /* Fork new server. */ int server_start(struct tmuxproc *client, struct event_base *base, int lockfd, @@ -217,7 +205,8 @@ server_start(struct tmuxproc *client, struct event_base *base, int lockfd, } if (cause != NULL) { - cmdq_append(c, cmdq_get_callback(server_start_error, cause)); + cmdq_append(c, cmdq_get_error(cause)); + free(cause); c->flags |= CLIENT_EXIT; } diff --git a/tmux.h b/tmux.h index 4317bf0b..c87c797c 100644 --- a/tmux.h +++ b/tmux.h @@ -1983,6 +1983,7 @@ struct cmdq_item *cmdq_get_command(struct cmd_list *, struct cmd_find_state *, struct mouse_event *, int); #define cmdq_get_callback(cb, data) cmdq_get_callback1(#cb, cb, data) struct cmdq_item *cmdq_get_callback1(const char *, cmdq_cb, void *); +struct cmdq_item *cmdq_get_error(const char *); void cmdq_insert_after(struct cmdq_item *, struct cmdq_item *); void cmdq_append(struct client *, struct cmdq_item *); void cmdq_insert_hook(struct session *, struct cmdq_item *, -- cgit From 87d82170a615007bbd2f880765f222dd5c263000 Mon Sep 17 00:00:00 2001 From: nicm Date: Mon, 20 May 2019 13:23:32 +0000 Subject: Fix ordering of source-file with multiple files and add flags to load_cfg. --- cfg.c | 25 ++++++++++++++----------- cmd-source-file.c | 14 ++++++++++---- tmux.h | 4 +++- 3 files changed, 27 insertions(+), 16 deletions(-) diff --git a/cfg.c b/cfg.c index 1b85ac95..0b74ec35 100644 --- a/cfg.c +++ b/cfg.c @@ -82,7 +82,7 @@ void start_cfg(void) { const char *home; - int quiet = 0; + int flags = 0; struct client *c; /* @@ -103,14 +103,14 @@ start_cfg(void) } if (cfg_file == NULL) - load_cfg(TMUX_CONF, NULL, NULL, 1); + load_cfg(TMUX_CONF, NULL, NULL, CFG_QUIET, NULL); if (cfg_file == NULL && (home = find_home()) != NULL) { xasprintf(&cfg_file, "%s/.tmux.conf", home); - quiet = 1; + flags = CFG_QUIET; } if (cfg_file != NULL) - load_cfg(cfg_file, NULL, NULL, quiet); + load_cfg(cfg_file, NULL, NULL, flags, NULL); cmdq_append(NULL, cmdq_get_callback(cfg_done, NULL)); } @@ -239,7 +239,8 @@ cfg_handle_directive(const char *p, const char *path, size_t line, } int -load_cfg(const char *path, struct client *c, struct cmdq_item *item, int quiet) +load_cfg(const char *path, struct client *c, struct cmdq_item *item, int flags, + struct cmdq_item **new_item) { FILE *f; const char delim[3] = { '\\', '\\', '\0' }; @@ -247,7 +248,7 @@ load_cfg(const char *path, struct client *c, struct cmdq_item *item, int quiet) size_t line = 0; char *buf, *cause1, *p, *q; struct cmd_list *cmdlist; - struct cmdq_item *new_item; + struct cmdq_item *new_item0; struct cfg_cond *cond, *cond1; struct cfg_conds conds; struct cmd_find_state *fs = NULL; @@ -262,7 +263,7 @@ load_cfg(const char *path, struct client *c, struct cmdq_item *item, int quiet) log_debug("loading %s", path); if ((f = fopen(path, "rb")) == NULL) { - if (errno == ENOENT && quiet) + if (errno == ENOENT && (flags & CFG_QUIET)) return (0); cfg_add_cause("%s: %s", path, strerror(errno)); return (-1); @@ -301,12 +302,12 @@ load_cfg(const char *path, struct client *c, struct cmdq_item *item, int quiet) } free(buf); - new_item = cmdq_get_command(cmdlist, NULL, NULL, 0); + new_item0 = cmdq_get_command(cmdlist, NULL, NULL, 0); if (item != NULL) { - cmdq_insert_after(item, new_item); - item = new_item; + cmdq_insert_after(item, new_item0); + item = new_item0; } else - cmdq_append(c, new_item); + cmdq_append(c, new_item0); cmd_list_free(cmdlist); found++; @@ -319,6 +320,8 @@ load_cfg(const char *path, struct client *c, struct cmdq_item *item, int quiet) free(cond); } + if (new_item != NULL) + *new_item = item; return (found); } diff --git a/cmd-source-file.c b/cmd-source-file.c index ffb7f423..fbe01c21 100644 --- a/cmd-source-file.c +++ b/cmd-source-file.c @@ -49,15 +49,18 @@ static enum cmd_retval cmd_source_file_exec(struct cmd *self, struct cmdq_item *item) { struct args *args = self->args; - int quiet = args_has(args, 'q'); + int flags = 0; struct client *c = item->client; - struct cmdq_item *new_item; + struct cmdq_item *new_item, *after; enum cmd_retval retval; char *pattern, *tmp; const char *path = args->argv[0]; glob_t g; u_int i; + if (args_has(args, 'q')) + flags |= CFG_QUIET; + if (*path == '/') pattern = xstrdup(path); else { @@ -69,7 +72,7 @@ cmd_source_file_exec(struct cmd *self, struct cmdq_item *item) retval = CMD_RETURN_NORMAL; if (glob(pattern, 0, NULL, &g) != 0) { - if (!quiet || errno != ENOENT) { + if (errno != ENOENT || (~flags & CFG_QUIET)) { cmdq_error(item, "%s: %s", path, strerror(errno)); retval = CMD_RETURN_ERROR; } @@ -78,9 +81,12 @@ cmd_source_file_exec(struct cmd *self, struct cmdq_item *item) } free(pattern); + after = item; for (i = 0; i < (u_int)g.gl_pathc; i++) { - if (load_cfg(g.gl_pathv[i], c, item, quiet) < 0) + if (load_cfg(g.gl_pathv[i], c, after, flags, &new_item) < 0) retval = CMD_RETURN_ERROR; + else if (new_item != NULL) + after = new_item; } if (cfg_finished) { new_item = cmdq_get_callback(cmd_source_file_done, NULL); diff --git a/tmux.h b/tmux.h index c87c797c..2979a6e3 100644 --- a/tmux.h +++ b/tmux.h @@ -1671,8 +1671,10 @@ void proc_toggle_log(struct tmuxproc *); /* cfg.c */ extern int cfg_finished; extern struct client *cfg_client; +#define CFG_QUIET 0x1 void start_cfg(void); -int load_cfg(const char *, struct client *, struct cmdq_item *, int); +int load_cfg(const char *, struct client *, struct cmdq_item *, int, + struct cmdq_item **); void set_cfg_file(const char *); void printflike(1, 2) cfg_add_cause(const char *, ...); void cfg_print_causes(struct cmdq_item *); -- cgit