From 21f94c6aa8f7753b6ba8e738bcdcf29567e3f7f9 Mon Sep 17 00:00:00 2001 From: okan Date: Wed, 14 Sep 2016 19:45:33 +0000 Subject: Fix-up a few simple uses of client_current(): check CLIENT_ACTIVE flag instead of relying on curcc. --- client.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'client.c') diff --git a/client.c b/client.c index a274c64..9dea242 100644 --- a/client.c +++ b/client.c @@ -15,7 +15,7 @@ * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. * - * $OpenBSD: client.c,v 1.219 2016/09/13 17:42:58 okan Exp $ + * $OpenBSD: client.c,v 1.220 2016/09/14 19:45:33 okan Exp $ */ #include @@ -176,12 +176,12 @@ client_delete(struct client_ctx *cc) xu_ewmh_net_client_list(sc); xu_ewmh_net_client_list_stacking(sc); + if (cc->flags & CLIENT_ACTIVE) + client_none(sc); + if (cc->gc != NULL) TAILQ_REMOVE(&cc->gc->clientq, cc, group_entry); - if (cc == client_current()) - client_none(sc); - while ((wn = TAILQ_FIRST(&cc->nameq)) != NULL) { TAILQ_REMOVE(&cc->nameq, wn, entry); free(wn->name); @@ -510,12 +510,12 @@ client_hide(struct client_ctx *cc) { XUnmapWindow(X_Dpy, cc->win); + if (cc->flags & CLIENT_ACTIVE) + client_none(cc->sc); + cc->flags &= ~CLIENT_ACTIVE; cc->flags |= CLIENT_HIDDEN; client_set_wm_state(cc, IconicState); - - if (cc == client_current()) - client_none(cc->sc); } void -- cgit v1.2.3-2-gb3c3