diff options
author | nicm <nicm> | 2019-05-25 06:58:10 +0000 |
---|---|---|
committer | nicm <nicm> | 2019-05-25 06:58:10 +0000 |
commit | 930245d7ffde91464865122a82437209680458c8 (patch) | |
tree | 40775b1dbe7d7dc6d2ec67f1638bbcac9bc1e26f | |
parent | 0dc8b7d5d8a3607de11732f660aa64aaf2844d11 (diff) | |
download | rtmux-930245d7ffde91464865122a82437209680458c8.tar.gz rtmux-930245d7ffde91464865122a82437209680458c8.tar.bz2 rtmux-930245d7ffde91464865122a82437209680458c8.zip |
Make cmd_log_argv take a printf-like format for the prefix.
-rw-r--r-- | cmd.c | 19 | ||||
-rw-r--r-- | spawn.c | 2 | ||||
-rw-r--r-- | tmux.h | 2 |
3 files changed, 15 insertions, 8 deletions
@@ -205,13 +205,20 @@ const struct cmd_entry *cmd_table[] = { NULL }; -void -cmd_log_argv(int argc, char **argv, const char *prefix) +void printflike(3, 4) +cmd_log_argv(int argc, char **argv, const char *fmt, ...) { - int i; + char *prefix; + va_list ap; + int i; + + va_start(ap, fmt); + xvasprintf(&prefix, fmt, ap); + va_end(ap); for (i = 0; i < argc; i++) log_debug("%s: argv[%d]=%s", prefix, i, argv[i]); + free(prefix); } void @@ -245,7 +252,7 @@ cmd_pack_argv(int argc, char **argv, char *buf, size_t len) if (argc == 0) return (0); - cmd_log_argv(argc, argv, __func__); + cmd_log_argv(argc, argv, "%s", __func__); *buf = '\0'; for (i = 0; i < argc; i++) { @@ -282,7 +289,7 @@ cmd_unpack_argv(char *buf, size_t len, int argc, char ***argv) buf += arglen; len -= arglen; } - cmd_log_argv(argc, *argv, __func__); + cmd_log_argv(argc, *argv, "%s", __func__); return (0); } @@ -437,7 +444,7 @@ cmd_parse(int argc, char **argv, const char *file, u_int line, char **cause) entry = cmd_find(name, cause); if (entry == NULL) return (NULL); - cmd_log_argv(argc, argv, entry->name); + cmd_log_argv(argc, argv, "%s: %s", __func__, entry->name); args = args_parse(entry->args.template, argc, argv); if (args == NULL) @@ -329,7 +329,7 @@ spawn_pane(struct spawn_context *sc, char **cause) } if (cwd != NULL) log_debug("%s: cwd=%s", __func__, cwd); - cmd_log_argv(new_wp->argc, new_wp->argv, __func__); + cmd_log_argv(new_wp->argc, new_wp->argv, "%s", __func__); environ_log(child, "%s: environment ", __func__); /* If the command is empty, don't fork a child process. */ @@ -1987,7 +1987,7 @@ int cmd_find_from_mouse(struct cmd_find_state *, int cmd_find_from_nothing(struct cmd_find_state *, int); /* cmd.c */ -void cmd_log_argv(int, char **, const char *); +void printflike(3, 4) cmd_log_argv(int, char **, const char *, ...); void cmd_prepend_argv(int *, char ***, char *); void cmd_append_argv(int *, char ***, char *); int cmd_pack_argv(int, char **, char *, size_t); |