diff options
author | Nicholas Marriott <nicm@openbsd.org> | 2013-03-24 09:27:19 +0000 |
---|---|---|
committer | Nicholas Marriott <nicm@openbsd.org> | 2013-03-24 09:27:19 +0000 |
commit | bb8457b166635bc8d069012bb33d94e44178bfdc (patch) | |
tree | fb327f1e877f6663043d8b8bb66b0513cb2146cb /cmd.c | |
parent | 3eae71b5b28cf591f6cff652e2f2c67be394c040 (diff) | |
download | rtmux-bb8457b166635bc8d069012bb33d94e44178bfdc.tar.gz rtmux-bb8457b166635bc8d069012bb33d94e44178bfdc.tar.bz2 rtmux-bb8457b166635bc8d069012bb33d94e44178bfdc.zip |
Fix error reporting for client commands by adding a flag to
cmd_find_client to tell it whether or not to show errors, sometimes it's
needed and sometimes not.
Diffstat (limited to 'cmd.c')
-rw-r--r-- | cmd.c | 12 |
1 files changed, 8 insertions, 4 deletions
@@ -517,15 +517,19 @@ cmd_choose_client(struct clients *cc) /* Find the target client or report an error and return NULL. */ struct client * -cmd_find_client(struct cmd_ctx *ctx, const char *arg) +cmd_find_client(struct cmd_ctx *ctx, const char *arg, int quiet) { struct client *c; char *tmparg; size_t arglen; /* A NULL argument means the current client. */ - if (arg == NULL) - return (cmd_current_client(ctx)); + if (arg == NULL) { + c = cmd_current_client(ctx); + if (c == NULL && !quiet) + ctx->error(ctx, "no clients"); + return (c); + } tmparg = xstrdup(arg); /* Trim a single trailing colon if any. */ @@ -537,7 +541,7 @@ cmd_find_client(struct cmd_ctx *ctx, const char *arg) c = cmd_lookup_client(tmparg); /* If no client found, report an error. */ - if (c == NULL) + if (c == NULL && !quiet) ctx->error(ctx, "client not found: %s", tmparg); free(tmparg); |