From 27b023ebcbe1f4ec24c6044fee51c22c55d5993b Mon Sep 17 00:00:00 2001
From: oga <oga>
Date: Sun, 18 May 2008 19:43:50 +0000
Subject: Kill conf_get_int(), it was a silly function anyway.

Since it's only used once just put the (simplified) logic into
conf_client() instead. This  means we can kill an enum and
CONF_IGNORECASE, too.

ok okan@
---
 calmwm.h | 14 +-------------
 conf.c   | 44 +++++++++-----------------------------------
 parse.y  |  1 -
 3 files changed, 10 insertions(+), 49 deletions(-)

diff --git a/calmwm.h b/calmwm.h
index 4f67369..5c06257 100644
--- a/calmwm.h
+++ b/calmwm.h
@@ -32,10 +32,6 @@
 
 #define	CONFFILE	".cwmrc"
 
-enum conftype {
-	CONF_BWIDTH, CONF_IGNORE,
-};
-
 #define ChildMask	(SubstructureRedirectMask|SubstructureNotifyMask)
 #define ButtonMask	(ButtonPressMask|ButtonReleaseMask)
 #define MouseMask	(ButtonMask|PointerMotionMask)
@@ -215,12 +211,9 @@ TAILQ_HEAD(xevent_q, xevent);
  * Match a window.
  */
 #define CONF_MAX_WINTITLE	256
-#define CONF_IGNORECASE		0x01
 struct winmatch {
 	TAILQ_ENTRY(winmatch)	entry;
-
-	char	title[CONF_MAX_WINTITLE];
-	int	opts;
+	char			title[CONF_MAX_WINTITLE];
 };
 
 TAILQ_HEAD(winmatch_q, winmatch);
@@ -422,16 +415,11 @@ struct screen_ctx	*screen_current(void);
 void			 screen_updatestackingorder(void);
 
 void			 conf_setup(struct conf *, const char *);
-int			 conf_get_int(struct client_ctx *, enum conftype);
 void			 conf_client(struct client_ctx *);
-void			 conf_bindkey(struct conf *,
-			     void (*)(struct client_ctx *, void *),
-			     int, int, int, void *);
 void			 conf_bindname(struct conf *, char *, char *);
 void			 conf_unbind(struct conf *, struct keybinding *);
 int			 conf_changed(char *);
 void			 conf_reload(struct conf *);
-char			*conf_get_str(struct client_ctx *, enum conftype);
 
 void			 kbfunc_client_lower(struct client_ctx *, void *);
 void			 kbfunc_client_raise(struct client_ctx *, void *);
diff --git a/conf.c b/conf.c
index fe14817..f595ff1 100644
--- a/conf.c
+++ b/conf.c
@@ -174,53 +174,27 @@ conf_setup(struct conf *c, const char *conffile)
 	(void)parse_config(c->conf_path, c);
 }
 
-int
-conf_get_int(struct client_ctx *cc, enum conftype ctype)
+void
+conf_client(struct client_ctx *cc)
 {
-	int val = -1, ignore = 0;
-	char *wname;
-	struct winmatch *wm;
-
-	wname = cc->name;
+	struct winmatch	*wm;
+	char		*wname = cc->name;
+	int		 ignore = 0;
 
 	/* Can wname be NULL? */
-
 	if (wname != NULL) {
 		TAILQ_FOREACH(wm, &Conf.ignoreq, entry) {
-			int (*cmpfun)(const char *, const char *, size_t) =
-			    wm->opts & CONF_IGNORECASE ? strncasecmp : strncmp;
-			if ((*cmpfun)(wm->title, wname, strlen(wm->title)) == 0) {
+			if (strncasecmp(wm->title, wname, strlen(wm->title))
+			    == 0) {
 				ignore = 1;
 				break;
 			}
 		}
-
 	} else
 		ignore = 1;
 
-	switch (ctype) {
-	case CONF_BWIDTH:
-		/*
-		 * XXX this will be a list, specified in the
-		 * configuration file.
-		 */
-		val = ignore ? 0 : 3;
-		break;
-	case CONF_IGNORE:
-		val = ignore;
-		break;
-	default:
-		break;
-	}
-
-	return (val);
-}
-
-void
-conf_client(struct client_ctx *cc)
-{
-	cc->bwidth = conf_get_int(cc, CONF_BWIDTH);
-	cc->flags |= conf_get_int(cc, CONF_IGNORE) ? CLIENT_IGNORE : 0;
+	cc->bwidth = ignore ? 0 : 3;
+	cc->flags |= ignore ? CLIENT_IGNORE : 0;
 }
 
 struct {
diff --git a/parse.y b/parse.y
index b9b9222..cf75142 100644
--- a/parse.y
+++ b/parse.y
@@ -152,7 +152,6 @@ main		: FONTNAME STRING		{
 
 			XCALLOC(wm, struct winmatch);
 			strlcpy(wm->title, $2, sizeof(wm->title));
-			wm->opts |= CONF_IGNORECASE;
 			TAILQ_INSERT_TAIL(&conf->ignoreq, wm, entry);
 
 			free($2);
-- 
cgit 1.4.1