tweaks: use a symbol instead of a hard-coded number
parent
fb5eb4856b
commit
2314a1932f
|
@ -991,9 +991,10 @@ void parse_includes(char *ptr)
|
||||||
free(expanded);
|
free(expanded);
|
||||||
}
|
}
|
||||||
|
|
||||||
const char hues[9][7] = { "pink", "purple", "mauve", "lagoon", "mint",
|
#define COLORCOUNT 9
|
||||||
|
const char hues[COLORCOUNT][7] = { "pink", "purple", "mauve", "lagoon", "mint",
|
||||||
"lime", "peach", "orange", "latte" };
|
"lime", "peach", "orange", "latte" };
|
||||||
short indices[9] = { 204, 163, 134, 38, 48, 148, 215, 208, 137 };
|
short indices[COLORCOUNT] = { 204, 163, 134, 38, 48, 148, 215, 208, 137 };
|
||||||
|
|
||||||
/* Return the short value corresponding to the given color name, and set
|
/* Return the short value corresponding to the given color name, and set
|
||||||
* vivid to TRUE for a lighter color, and thick for a heavier typeface. */
|
* vivid to TRUE for a lighter color, and thick for a heavier typeface. */
|
||||||
|
@ -1032,7 +1033,7 @@ short color_to_short(const char *colorname, bool *vivid, bool *thick)
|
||||||
else if (strcmp(colorname, "normal") == 0)
|
else if (strcmp(colorname, "normal") == 0)
|
||||||
return THE_DEFAULT;
|
return THE_DEFAULT;
|
||||||
else
|
else
|
||||||
for (int index = 0; index < 9; index++)
|
for (int index = 0; index < COLORCOUNT; index++)
|
||||||
if (strcmp(colorname, hues[index]) == 0) {
|
if (strcmp(colorname, hues[index]) == 0) {
|
||||||
if (*vivid) {
|
if (*vivid) {
|
||||||
jot_error(N_("Color '%s' takes no prefix"), colorname);
|
jot_error(N_("Color '%s' takes no prefix"), colorname);
|
||||||
|
|
Loading…
Reference in New Issue