diff options
| author | Okan Demirmen <okan@cvs.openbsd.org> | 2019-02-28 23:26:13 +0000 |
|---|---|---|
| committer | Okan Demirmen <okan@cvs.openbsd.org> | 2019-02-28 23:26:13 +0000 |
| commit | d6c519554efc91144b6187b3c4ab21ea0ce64402 (patch) | |
| tree | 087fbae08857a08f722de2d931d393ea68b33e60 /app/cwm/group.c | |
| parent | e01fb8dcc37107b531022e67db303f5723ca93d4 (diff) | |
Move the group index (desktop number) check to the only 2 callers that require
checking due to ewmh.
Diffstat (limited to 'app/cwm/group.c')
| -rw-r--r-- | app/cwm/group.c | 14 |
1 files changed, 1 insertions, 13 deletions
diff --git a/app/cwm/group.c b/app/cwm/group.c index 06e73eb8d..c51bf5124 100644 --- a/app/cwm/group.c +++ b/app/cwm/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.131 2019/02/28 23:20:52 okan Exp $ + * $OpenBSD: group.c,v 1.132 2019/02/28 23:26:12 okan Exp $ */ #include <sys/types.h> @@ -156,9 +156,6 @@ group_movetogroup(struct client_ctx *cc, int idx) struct screen_ctx *sc = cc->sc; struct group_ctx *gc; - if (idx < 0 || idx >= Conf.ngroups) - return; - TAILQ_FOREACH(gc, &sc->groupq, entry) { if (gc->num == idx) { if (cc->gc == gc) @@ -216,9 +213,6 @@ group_hidetoggle(struct screen_ctx *sc, int idx) { struct group_ctx *gc; - if (idx < 0 || idx >= Conf.ngroups) - return; - TAILQ_FOREACH(gc, &sc->groupq, entry) { if (gc->num == idx) { if (group_holds_only_hidden(gc)) @@ -238,9 +232,6 @@ group_only(struct screen_ctx *sc, int idx) { struct group_ctx *gc; - if (idx < 0 || idx >= Conf.ngroups) - return; - TAILQ_FOREACH(gc, &sc->groupq, entry) { if (gc->num == idx) group_show(gc); @@ -255,9 +246,6 @@ group_close(struct screen_ctx *sc, int idx) struct group_ctx *gc; struct client_ctx *cc; - if (idx < 0 || idx >= Conf.ngroups) - return; - TAILQ_FOREACH(gc, &sc->groupq, entry) { if (gc->num == idx) { TAILQ_FOREACH(cc, &gc->clientq, group_entry) |