aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorokan2015-06-28 19:50:46 +0000
committerokan2015-06-28 19:50:46 +0000
commit8d83067e01a0b43472f48c16ce94d2ecfe015af4 (patch)
tree218e7e5bdb91a43427a67c689a5093db4225135d
parent603997932d26636c5a55d4efd9cf980f86b78088 (diff)
downloadcwm-8d83067e01a0b43472f48c16ce94d2ecfe015af4.tar.gz
replace assert usage
-rw-r--r--client.c7
-rw-r--r--group.c7
-rw-r--r--search.c6
3 files changed, 9 insertions, 11 deletions
diff --git a/client.c b/client.c
index 857ff8e..94f00f9 100644
--- a/client.c
+++ b/client.c
@@ -15,13 +15,12 @@
* 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.194 2015/06/26 17:17:46 okan Exp $
+ * $OpenBSD: client.c,v 1.195 2015/06/28 19:50:46 okan Exp $
*/
#include <sys/types.h>
#include <sys/queue.h>
-#include <assert.h>
#include <err.h>
#include <errno.h>
#include <limits.h>
@@ -635,8 +634,8 @@ match:
/* Now, do some garbage collection. */
if (cc->nameqlen > CLIENT_MAXNAMEQLEN) {
- wn = TAILQ_FIRST(&cc->nameq);
- assert(wn != NULL);
+ if ((wn = TAILQ_FIRST(&cc->nameq)) == NULL)
+ errx(1, "client_setname: window name queue empty");
TAILQ_REMOVE(&cc->nameq, wn, entry);
free(wn->name);
free(wn);
diff --git a/group.c b/group.c
index d18ece8..d2c0bf8 100644
--- a/group.c
+++ b/group.c
@@ -16,13 +16,12 @@
* 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.111 2015/03/28 23:12:47 okan Exp $
+ * $OpenBSD: group.c,v 1.112 2015/06/28 19:50:46 okan Exp $
*/
#include <sys/types.h>
#include <sys/queue.h>
-#include <assert.h>
#include <err.h>
#include <errno.h>
#include <limits.h>
@@ -264,9 +263,9 @@ group_cycle(struct screen_ctx *sc, int flags)
{
struct group_ctx *gc, *showgroup = NULL;
- assert(sc->group_active != NULL);
+ if (((gc = sc->group_active)) == NULL)
+ errx(1, "group_cycle: no active group");
- gc = sc->group_active;
for (;;) {
gc = (flags & CWM_RCYCLE) ? TAILQ_PREV(gc, group_ctx_q,
entry) : TAILQ_NEXT(gc, entry);
diff --git a/search.c b/search.c
index 44f233e..deff8e3 100644
--- a/search.c
+++ b/search.c
@@ -15,13 +15,12 @@
* ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*
- * $OpenBSD: search.c,v 1.43 2015/06/08 15:34:03 okan Exp $
+ * $OpenBSD: search.c,v 1.44 2015/06/28 19:50:46 okan Exp $
*/
#include <sys/types.h>
#include <sys/queue.h>
-#include <assert.h>
#include <err.h>
#include <errno.h>
#include <fnmatch.h>
@@ -106,7 +105,8 @@ search_match_client(struct menu_q *menuq, struct menu_q *resultq, char *search)
if ((cc->flags & CLIENT_HIDDEN) && (tier > 0))
tier--;
- assert(tier < nitems(tierp));
+ if (tier >= nitems(tierp))
+ errx(1, "search_match_client: invalid tier");
/*
* If you have a tierp, insert after it, and make it