From faadfe2b93db0976952b13a7870074ac8979bcd9 Mon Sep 17 00:00:00 2001 From: Harald Welte Date: Fri, 26 Mar 2010 21:05:43 +0800 Subject: debug: remove unneeded 'number' member of 'struct debug_info_cat' As the debug subsystem number is used as index into the debug_info_cat array, there is no need to store the number explicitly inside the structure again. --- src/debug.c | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) (limited to 'src') diff --git a/src/debug.c b/src/debug.c index d10861f8..19fc3058 100644 --- a/src/debug.c +++ b/src/debug.c @@ -59,7 +59,7 @@ int debug_parse_category(const char *category) for (i = 0; i < debug_info->num_cat; ++i) { if (!strcasecmp(debug_info->cat[i].name+1, category)) - return debug_info->cat[i].number; + return i; } return -EINVAL; @@ -91,14 +91,13 @@ void debug_parse_category_mask(struct debug_target* target, const char *_mask) if (strncasecmp(debug_info->cat[i].name, category_token, length) == 0) { - int number = debug_info->cat[i].number; int level = 0; if (colon) level = atoi(colon+1); - target->categories[number].enabled = 1; - target->categories[number].loglevel = level; + target->categories[i].enabled = 1; + target->categories[i].loglevel = level; } } } while ((category_token = strtok(NULL, ":"))); @@ -108,12 +107,8 @@ void debug_parse_category_mask(struct debug_target* target, const char *_mask) static const char* color(int subsys) { - int i = 0; - - for (i = 0; i < debug_info->num_cat; ++i) { - if (debug_info->cat[i].number == subsys) - return debug_info->cat[i].color; - } + if (subsys < debug_info->num_cat) + return debug_info->cat[subsys].color; return NULL; } -- cgit v1.2.3