From bd1acc3e94f8002d5b5796f67eac6eb6755dcceb Mon Sep 17 00:00:00 2001
From: okan
Date: Tue, 25 Aug 2015 19:49:19 +0000
Subject: Allowing sending a valid 'nogroup' (0) group_ctx to group_assign()
 (since we init all groups), though assigning the client's group to NULL for
 'sticky'; use this simplification in a few places (others to follow).

---
 group.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/group.c b/group.c
index 593a0d6..19b5963 100644
--- a/group.c
+++ b/group.c
@@ -16,7 +16,7 @@
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  *
- * $OpenBSD: group.c,v 1.115 2015/08/25 18:29:10 okan Exp $
+ * $OpenBSD: group.c,v 1.116 2015/08/25 19:49:19 okan Exp $
  */
 
 #include <sys/types.h>
@@ -48,6 +48,9 @@ group_assign(struct group_ctx *gc, struct client_ctx *cc)
 	if (cc->group != NULL)
 		TAILQ_REMOVE(&cc->group->clientq, cc, group_entry);
 
+	if ((gc != NULL) && (gc->num == 0))
+		gc = NULL;
+
 	cc->group = gc;
 
 	if (cc->group != NULL)
@@ -129,6 +132,8 @@ group_init(struct screen_ctx *sc, int num)
 	gc->num = num;
 	TAILQ_INIT(&gc->clientq);
 
+	fprintf(stderr, "%d: %s\n", gc->num, gc->name);
+
 	TAILQ_INSERT_TAIL(&sc->groupq, gc, entry);
 
 	if (num == 1)
@@ -338,7 +343,7 @@ group_restore(struct client_ctx *cc)
 	num = MIN(*grpnum, (CALMWM_NGROUPS - 1));
 	XFree(grpnum);
 
-	if ((num == -1) || (num == 0)) {
+	if (num == -1) {
 		group_assign(NULL, cc);
 		return(1);
 	}
@@ -373,10 +378,6 @@ group_autogroup(struct client_ctx *cc)
 		}
 	}
 
-	if (num == 0) {
-		group_assign(NULL, cc);
-		return(1);
-	}
 	TAILQ_FOREACH(gc, &sc->groupq, entry) {
 		if (gc->num == num) {
 			group_assign(gc, cc);
-- 
cgit v1.2.3-2-gb3c3