Add a windowonly generic command and use it where appropriate. Also trim includes and unused.

1 files changed, 13 insertions, 111 deletions

diff --git a/cmd-unlink-window.c b/cmd-unlink-window.c
index e1c88f23..d44f3d9f 100644
--- a/cmd-unlink-window.c
+++ b/cmd-unlink-window.c
@@ -1,4 +1,4 @@
-/* $Id: cmd-unlink-window.c,v 1.7 2008-06-02 21:08:36 nicm Exp $ */
+/* $Id: cmd-unlink-window.c,v 1.8 2008-06-02 22:09:49 nicm Exp $ */
 
 /*
  * Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net>
@@ -18,108 +18,38 @@
 
 #include <sys/types.h>
 
-#include <getopt.h>
-#include <stdlib.h>
-
 #include "tmux.h"
 
 /*
  * Unlink a window, unless it would be destroyed by doing so (only one link).
  */
 
-int	cmd_unlink_window_parse(struct cmd *, void **, int, char **, char **);
 void	cmd_unlink_window_exec(void *, struct cmd_ctx *);
-void	cmd_unlink_window_send(void *, struct buffer *);
-void	cmd_unlink_window_recv(void **, struct buffer *);
-void	cmd_unlink_window_free(void *);
-
-struct cmd_unlink_window_data {
-	char	*sname;
-	int	 idx;
-};
 
 const struct cmd_entry cmd_unlink_window_entry = {
 	"unlink-window", "unlinkw",
-	"[-i index] [-s session-name]",
+	CMD_WINDOWONLY_USAGE,
 	0,
-	cmd_unlink_window_parse,
+	cmd_windowonly_parse,
 	cmd_unlink_window_exec,
-	cmd_unlink_window_send,
-	cmd_unlink_window_recv,
-	cmd_unlink_window_free
-};
-
-int
-cmd_unlink_window_parse(
-    struct cmd *self, void **ptr, int argc, char **argv, char **cause)
-{
-	struct cmd_unlink_window_data	*data;
-	const char			*errstr;
-	int				 opt;
-
-	*ptr = data = xmalloc(sizeof *data);
-	data->sname = NULL;
-	data->idx = -1;
-
-	while ((opt = getopt(argc, argv, "i:s:")) != EOF) {
-		switch (opt) {
-		case 'i':
-			data->idx = strtonum(optarg, 0, INT_MAX, &errstr);
-			if (errstr != NULL) {
-				xasprintf(cause, "index %s", errstr);
-				goto error;
-			}
-			break;
-		case 's':
-			data->sname = xstrdup(optarg);
-			break;
-		default:
-			goto usage;
-		}
-	}
-	argc -= optind;
-	argv += optind;
-	if (argc != 0)
-		goto usage;
-
-	return (0);
+	cmd_windowonly_send,
+	cmd_windowonly_recv,
+	cmd_windowonly_free
 
-usage:
-	xasprintf(cause, "usage: %s %s", self->entry->name, self->entry->usage);
-
-error:
-	cmd_unlink_window_free(data);
-	return (-1);
-}
+};
 
 void
 cmd_unlink_window_exec(void *ptr, struct cmd_ctx *ctx)
 {
-	struct cmd_unlink_window_data	*data = ptr;
-	struct session			*s;
-	struct client			*c;
-	struct winlink			*wl;
-	u_int		 		 i;
-	int		 		 destroyed;
-
-	if (data == NULL)
-		return;
+	struct winlink	*wl;
+	struct session	*s;
+	struct client	*c;
+	u_int		 i;
+	int		 destroyed;
 
-	if ((s = cmd_find_session(ctx, data->sname)) == NULL)
+	if ((wl = cmd_windowonly_get(ptr, ctx, &s)) == NULL)
 		return;
 
-	if (data->idx < 0)
-		data->idx = -1;
-	if (data->idx == -1)
-		wl = s->curw;
-	else {
-		wl = winlink_find_by_index(&s->windows, data->idx);
-		if (wl == NULL) {
-			ctx->error(ctx, "no window %d", data->idx);
-			return;
-		}
-	}
-
 	if (wl->window->references == 1) {
 		ctx->error(ctx, "window is only linked to one session");
 		return;
@@ -140,31 +70,3 @@ cmd_unlink_window_exec(void *ptr, struct cmd_ctx *ctx)
 	if (ctx->cmdclient != NULL)
 		server_write_client(ctx->cmdclient, MSG_EXIT, NULL, 0);
 }
-
-void
-cmd_unlink_window_send(void *ptr, struct buffer *b)
-{
-	struct cmd_unlink_window_data	*data = ptr;
-
-	buffer_write(b, data, sizeof *data);
-	cmd_send_string(b, data->sname);
-}
-
-void
-cmd_unlink_window_recv(void **ptr, struct buffer *b)
-{
-	struct cmd_unlink_window_data	*data;
-
-	*ptr = data = xmalloc(sizeof *data);
-	buffer_read(b, data, sizeof *data);
-}
-
-void
-cmd_unlink_window_free(void *ptr)
-{
-	struct cmd_unlink_window_data	*data = ptr;
-
-	if (data->sname != NULL)
-		xfree(data->sname);
-	xfree(data);
-}