From 57d50a524c379e824182da91ab5d7ba0d1551072 Mon Sep 17 00:00:00 2001
From: nicm <nicm@openbsd.org>
Date: Thu, 8 Jun 2017 07:48:04 +0000
Subject: [PATCH] Add size to client descriptions in list, suggested by Greg
 Hurrell.

---
 usr.bin/tmux/window-client.c | 32 ++++++++++++++++++++++++++++----
 1 file changed, 28 insertions(+), 4 deletions(-)

diff --git a/usr.bin/tmux/window-client.c b/usr.bin/tmux/window-client.c
index 6cc2b0574bc..3aa67ef3f63 100644
--- a/usr.bin/tmux/window-client.c
+++ b/usr.bin/tmux/window-client.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: window-client.c,v 1.3 2017/06/07 14:37:30 nicm Exp $ */
+/* $OpenBSD: window-client.c,v 1.4 2017/06/08 07:48:04 nicm Exp $ */
 
 /*
  * Copyright (c) 2017 Nicholas Marriott <nicholas.marriott@gmail.com>
@@ -47,11 +47,13 @@ const struct window_mode window_client_mode = {
 
 enum window_client_sort_type {
 	WINDOW_CLIENT_BY_NAME,
+	WINDOW_CLIENT_BY_SIZE,
 	WINDOW_CLIENT_BY_CREATION_TIME,
 	WINDOW_CLIENT_BY_ACTIVITY_TIME,
 };
 static const char *window_client_sort_list[] = {
 	"name",
+	"size",
 	"creation time",
 	"activity time"
 };
@@ -95,6 +97,23 @@ window_client_cmp_name(const void *a0, const void *b0)
 	return (strcmp((*a)->c->name, (*b)->c->name));
 }
 
+static int
+window_client_cmp_size(const void *a0, const void *b0)
+{
+	const struct window_client_itemdata *const *a = a0;
+	const struct window_client_itemdata *const *b = b0;
+
+	if ((*a)->c->tty.sx < (*b)->c->tty.sx)
+		return (-1);
+	if ((*a)->c->tty.sx > (*b)->c->tty.sx)
+		return (1);
+	if ((*a)->c->tty.sy < (*b)->c->tty.sy)
+		return (-1);
+	if ((*a)->c->tty.sy > (*b)->c->tty.sy)
+		return (1);
+	return (strcmp((*a)->c->name, (*b)->c->name));
+}
+
 static int
 window_client_cmp_creation_time(const void *a0, const void *b0)
 {
@@ -105,7 +124,7 @@ window_client_cmp_creation_time(const void *a0, const void *b0)
 		return (-1);
 	if (timercmp(&(*a)->c->creation_time, &(*b)->c->creation_time, <))
 		return (1);
-	return (0);
+	return (strcmp((*a)->c->name, (*b)->c->name));
 }
 
 static int
@@ -118,7 +137,7 @@ window_client_cmp_activity_time(const void *a0, const void *b0)
 		return (-1);
 	if (timercmp(&(*a)->c->activity_time, &(*b)->c->activity_time, <))
 		return (1);
-	return (0);
+	return (strcmp((*a)->c->name, (*b)->c->name));
 }
 
 static void
@@ -152,6 +171,10 @@ window_client_build(void *modedata, u_int sort_type, __unused uint64_t *tag)
 		qsort(data->item_list, data->item_size, sizeof *data->item_list,
 		    window_client_cmp_name);
 		break;
+	case WINDOW_CLIENT_BY_SIZE:
+		qsort(data->item_list, data->item_size, sizeof *data->item_list,
+		    window_client_cmp_size);
+		break;
 	case WINDOW_CLIENT_BY_CREATION_TIME:
 		qsort(data->item_list, data->item_size, sizeof *data->item_list,
 		    window_client_cmp_creation_time);
@@ -169,7 +192,8 @@ window_client_build(void *modedata, u_int sort_type, __unused uint64_t *tag)
 		tim = ctime(&c->activity_time.tv_sec);
 		*strchr(tim, '\n') = '\0';
 
-		xasprintf(&text, "session %s (%s)", c->session->name, tim);
+		xasprintf(&text, "session %s (%ux%u, %s)", c->session->name,
+		    c->tty.sx, c->tty.sy, tim);
 		mode_tree_add(data->data, NULL, item, (uint64_t)c, c->name,
 		    text, -1);
 		free(text);
-- 
2.20.1