error reporting for the rest of the color parsing

This commit is contained in:
Daniel Micay 2012-06-19 17:37:52 -04:00
parent 46f1aa9ea9
commit 5f9a4026e5
1 changed files with 13 additions and 8 deletions

View File

@ -293,7 +293,6 @@ static void load_config(GtkWindow *window, VteTerminal *vte,
static const char *filename = "termite.cfg"; static const char *filename = "termite.cfg";
const char *dir = g_get_user_config_dir(); const char *dir = g_get_user_config_dir();
char *path = g_build_filename(dir, filename, NULL); char *path = g_build_filename(dir, filename, NULL);
GKeyFile *config = g_key_file_new(); GKeyFile *config = g_key_file_new();
if ((g_key_file_load_from_file(config, path, G_KEY_FILE_NONE, NULL) || if ((g_key_file_load_from_file(config, path, G_KEY_FILE_NONE, NULL) ||
@ -387,7 +386,7 @@ static void load_config(GtkWindow *window, VteTerminal *vte,
vte_terminal_set_opacity(vte, (guint16)(0xffff * (1 - cfgdouble))); vte_terminal_set_opacity(vte, (guint16)(0xffff * (1 - cfgdouble)));
} }
GdkColor foreground, background, cursor, palette[16]; GdkColor color, palette[16];
static const char *colors[8] = {"black", "red", "green", "yellow", static const char *colors[8] = {"black", "red", "green", "yellow",
"blue", "magenta", "cyan", "white"}; "blue", "magenta", "cyan", "white"};
@ -417,22 +416,28 @@ static void load_config(GtkWindow *window, VteTerminal *vte,
} }
if (get_config_string(config, "colors", "foreground", &cfgstr)) { if (get_config_string(config, "colors", "foreground", &cfgstr)) {
if (gdk_color_parse(cfgstr, &foreground)) { if (gdk_color_parse(cfgstr, &color)) {
vte_terminal_set_color_foreground(vte, &foreground); vte_terminal_set_color_foreground(vte, &color);
} else {
g_printerr("invalid color string: %s\n", cfgstr);
} }
g_free(cfgstr); g_free(cfgstr);
} }
if (get_config_string(config, "colors", "background", &cfgstr)) { if (get_config_string(config, "colors", "background", &cfgstr)) {
if (gdk_color_parse(cfgstr, &background)) { if (gdk_color_parse(cfgstr, &color)) {
vte_terminal_set_color_background(vte, &background); vte_terminal_set_color_background(vte, &color);
} else {
g_printerr("invalid color string: %s\n", cfgstr);
} }
g_free(cfgstr); g_free(cfgstr);
} }
if (get_config_string(config, "colors", "cursor", &cfgstr)) { if (get_config_string(config, "colors", "cursor", &cfgstr)) {
if (gdk_color_parse(cfgstr, &cursor)) { if (gdk_color_parse(cfgstr, &color)) {
vte_terminal_set_color_cursor(vte, &cursor); vte_terminal_set_color_cursor(vte, &color);
} else {
g_printerr("invalid color string: %s\n", cfgstr);
} }
g_free(cfgstr); g_free(cfgstr);
} }