From: Max Kellermann <max@duempel.org>
Date: Fri, 3 Oct 2008 10:57:31 +0000 (+0200)
Subject: colors: added enum color_t
X-Git-Tag: v0.12_alpha1~88
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=43dfcc89822e46657db0d2bd52721e4ce852a357;p=ncmpc.git

colors: added enum color_t

Instead of declaring a bunch of CPP macros, use a C enum for
identifying colors.
---

diff --git a/src/colors.c b/src/colors.c
index 507be48..690b9f2 100644
--- a/src/colors.c
+++ b/src/colors.c
@@ -58,7 +58,7 @@ typedef struct {
 } color_definition_entry_t;
 
 typedef struct {
-	int id;
+	enum color id;
 	const char *name;
 	short fg;
 	attr_t attrs;
@@ -86,7 +86,7 @@ static short bg = COLOR_BLACK;
 static GList *color_definition_list = NULL;
 
 static color_entry_t *
-colors_lookup(int id)
+colors_lookup(enum color id)
 {
 	int i = 0;
 
@@ -114,7 +114,7 @@ colors_lookup_by_name(const char *name)
 }
 
 static int
-colors_update_pair(int id)
+colors_update_pair(enum color id)
 {
 	color_entry_t *entry = colors_lookup(id);
 	short fg = -1;
@@ -279,7 +279,7 @@ colors_start(void)
 }
 
 int
-colors_use(WINDOW *w, int id)
+colors_use(WINDOW *w, enum color id)
 {
 	color_entry_t *entry = colors_lookup(id);
 	short pair;
@@ -292,7 +292,7 @@ colors_use(WINDOW *w, int id)
 
 	if (options.enable_colors) {
 		/* color mode */
-		if (attrs != entry->attrs || id != pair)
+		if (attrs != entry->attrs || (short)id != pair)
 			wattr_set(w, entry->attrs, id, NULL);
 	} else {
 		/* mono mode */
diff --git a/src/colors.h b/src/colors.h
index f8a22d2..df03355 100644
--- a/src/colors.h
+++ b/src/colors.h
@@ -3,23 +3,25 @@
 
 #include <ncurses.h>
 
-#define COLOR_TITLE 1
-#define COLOR_TITLE_BOLD 2
-#define COLOR_LINE 3
-#define COLOR_LINE_BOLD 4
-#define COLOR_LIST 5
-#define COLOR_LIST_BOLD 6
-#define COLOR_PROGRESSBAR 7
-#define COLOR_STATUS 8
-#define COLOR_STATUS_BOLD 9
-#define COLOR_STATUS_TIME 10
-#define COLOR_STATUS_ALERT 11
+enum color {
+	COLOR_TITLE = 1,
+	COLOR_TITLE_BOLD,
+	COLOR_LINE,
+	COLOR_LINE_BOLD,
+	COLOR_LIST,
+	COLOR_LIST_BOLD,
+	COLOR_PROGRESSBAR,
+	COLOR_STATUS,
+	COLOR_STATUS_BOLD,
+	COLOR_STATUS_TIME,
+	COLOR_STATUS_ALERT,
+};
 
 short colors_str2color(const char *str);
 
 int colors_assign(const char *name, const char *value);
 int colors_define(const char *name, short r, short g, short b);
 int colors_start(void);
-int colors_use(WINDOW *w, int id);
+int colors_use(WINDOW *w, enum color id);
 
 #endif /* COLORS_H */