diff options
| author |   nicm <nicm@openbsd.org> | 2011-10-27 22:40:15 +0000 |
|---|---|---|
| committer | nicm <nicm@openbsd.org> | 2011-10-27 22:40:15 +0000 |
| commit | 12828c2bd37f0a2c6ee7cdf38dce6c761c7f0aa0 (patch) | |
| tree | 9956cf8b8e2b7ecc26dc754a83a5a70d272c0f45 | |
| parent | Use PRI{x,d}64 in format strings instead of %llx, %lld or %qd to print {u_,}int64_t or time_t (diff) | |
| download | wireguard-openbsd-12828c2bd37f0a2c6ee7cdf38dce6c761c7f0aa0.tar.xz wireguard-openbsd-12828c2bd37f0a2c6ee7cdf38dce6c761c7f0aa0.zip | |
Didn't really think the else behaviour through - requiring argv to
contain "else" is silly so just omit that, also some manpage
tweaks. From Tiago Cunha.
| -rw-r--r-- | usr.bin/tmux/cmd-if-shell.c | 28 | ||||
| -rw-r--r-- | usr.bin/tmux/tmux.1 | 12 |
2 files changed, 15 insertions, 25 deletions
diff --git a/usr.bin/tmux/cmd-if-shell.c b/usr.bin/tmux/cmd-if-shell.c index 45a5f9d8cc9..93c75f68705 100644 --- a/usr.bin/tmux/cmd-if-shell.c +++ b/usr.bin/tmux/cmd-if-shell.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cmd-if-shell.c,v 1.13 2011/10/18 08:57:01 nicm Exp $ */ +/* $OpenBSD: cmd-if-shell.c,v 1.14 2011/10/27 22:40:15 nicm Exp $ */ /* * Copyright (c) 2009 Tiago Cunha <me@tiagocunha.org> @@ -25,10 +25,9 @@ #include "tmux.h" /* - * Executes a tmux command if a shell command returns true. + * Executes a tmux command if a shell command returns true or false. */ -int cmd_if_shell_check(struct args *); int cmd_if_shell_exec(struct cmd *, struct cmd_ctx *); void cmd_if_shell_callback(struct job *); @@ -36,11 +35,11 @@ void cmd_if_shell_free(void *); const struct cmd_entry cmd_if_shell_entry = { "if-shell", "if", - "", 2, 4, - "shell-command command [else command]", + "", 2, 3, + "shell-command command [command]", 0, NULL, - cmd_if_shell_check, + NULL, cmd_if_shell_exec }; @@ -51,16 +50,6 @@ struct cmd_if_shell_data { }; int -cmd_if_shell_check(struct args *args) -{ - if (args->argc == 3) - return (-1); - if (args->argc == 4 && strcmp(args->argv[2], "else") != 0) - return (-1); - return (0); -} - -int cmd_if_shell_exec(struct cmd *self, struct cmd_ctx *ctx) { struct args *args = self->args; @@ -69,8 +58,8 @@ cmd_if_shell_exec(struct cmd *self, struct cmd_ctx *ctx) cdata = xmalloc(sizeof *cdata); cdata->cmd_if = xstrdup(args->argv[1]); - if (args->argc == 4) - cdata->cmd_else = xstrdup(args->argv[3]); + if (args->argc == 3) + cdata->cmd_else = xstrdup(args->argv[2]); else cdata->cmd_else = NULL; memcpy(&cdata->ctx, ctx, sizeof cdata->ctx); @@ -91,8 +80,7 @@ cmd_if_shell_callback(struct job *job) struct cmd_if_shell_data *cdata = job->data; struct cmd_ctx *ctx = &cdata->ctx; struct cmd_list *cmdlist; - char *cmd; - char *cause; + char *cause, *cmd; if (!WIFEXITED(job->status) || WEXITSTATUS(job->status) != 0) { cmd = cdata->cmd_else; diff --git a/usr.bin/tmux/tmux.1 b/usr.bin/tmux/tmux.1 index ac7c83d49f6..f64244f4cdc 100644 --- a/usr.bin/tmux/tmux.1 +++ b/usr.bin/tmux/tmux.1 @@ -1,4 +1,4 @@ -.\" $OpenBSD: tmux.1,v 1.250 2011/10/23 01:12:46 nicm Exp $ +.\" $OpenBSD: tmux.1,v 1.251 2011/10/27 22:40:15 nicm Exp $ .\" .\" Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net> .\" @@ -14,7 +14,7 @@ .\" IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING .\" OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. .\" -.Dd $Mdocdate: October 23 2011 $ +.Dd $Mdocdate: October 27 2011 $ .Dt TMUX 1 .Os .Sh NAME @@ -2971,13 +2971,15 @@ Miscellaneous commands are as follows: .Bl -tag -width Ds .It Ic clock-mode Op Fl t Ar target-pane Display a large clock. -.It Ic if-shell Ar shell-command command +.It Ic if-shell Ar shell-command command Op Ar command .D1 (alias: Ic if ) -Execute +Execute the first .Ar command if .Ar shell-command -returns success. +returns success or the second +.Ar command +otherwise. .It Ic lock-server .D1 (alias: Ic lock ) Lock each client individually by running the command specified by the |