diff options
| author | Lukas Fleischer <calcurse@cryptocrack.de> | 2011-07-19 15:35:20 +0200 |
|---|---|---|
| committer | Lukas Fleischer <calcurse@cryptocrack.de> | 2011-07-21 17:57:59 +0200 |
| commit | 32f22fdcc520f3ee28907eca7b8c155faf64c5d7 (patch) | |
| tree | 4217cfe1d8c1077d12cf051602bea818481c9be1 | |
| parent | 8e4238c242965758103cc893d0ca82b1a7c561bd (diff) | |
| download | calcurse-32f22fdcc520f3ee28907eca7b8c155faf64c5d7.tar.gz calcurse-32f22fdcc520f3ee28907eca7b8c155faf64c5d7.zip | |
Return error status in custom_set_conf()
Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>
| -rw-r--r-- | src/custom.c | 30 |
1 files changed, 16 insertions, 14 deletions
diff --git a/src/custom.c b/src/custom.c index 5821c9d..8f93fef 100644 --- a/src/custom.c +++ b/src/custom.c @@ -249,7 +249,7 @@ custom_remove_attr (WINDOW *win, int attr_num) } /* Set a configuration variable. */ -static void +static int custom_set_conf (struct conf *conf, enum conf_var var, char *val) { unsigned tmp; @@ -257,35 +257,35 @@ custom_set_conf (struct conf *conf, enum conf_var var, char *val) switch (var) { case CUSTOM_CONF_AUTOSAVE: - conf_parse_bool (&conf->auto_save, val); + return conf_parse_bool (&conf->auto_save, val); break; case CUSTOM_CONF_PERIODICSAVE: - conf_parse_unsigned (&conf->periodic_save, val); + return conf_parse_unsigned (&conf->periodic_save, val); break; case CUSTOM_CONF_CONFIRMQUIT: - conf_parse_bool (&conf->confirm_quit, val); + return conf_parse_bool (&conf->confirm_quit, val); break; case CUSTOM_CONF_CONFIRMDELETE: - conf_parse_bool (&conf->confirm_delete, val); + return conf_parse_bool (&conf->confirm_delete, val); break; case CUSTOM_CONF_SKIPSYSTEMDIALOGS: - conf_parse_bool (&conf->skip_system_dialogs, val); + return conf_parse_bool (&conf->skip_system_dialogs, val); break; case CUSTOM_CONF_SKIPPROGRESSBAR: - conf_parse_bool (&conf->skip_progress_bar, val); + return conf_parse_bool (&conf->skip_progress_bar, val); break; case CUSTOM_CONF_CALENDAR_DEFAULTVIEW: calendar_set_view (atoi (val)); break; case CUSTOM_CONF_WEEKBEGINSONMONDAY: - conf_parse_bool (&tmp, val); + return conf_parse_bool (&tmp, val); if (tmp) calendar_set_first_day_of_week (MONDAY); else calendar_set_first_day_of_week (SUNDAY); break; case CUSTOM_CONF_COLORTHEME: - conf_parse_color (val); + return conf_parse_color (val); break; case CUSTOM_CONF_LAYOUT: wins_set_layout (atoi (val)); @@ -294,7 +294,7 @@ custom_set_conf (struct conf *conf, enum conf_var var, char *val) wins_set_sbar_width (atoi (val)); break; case CUSTOM_CONF_NOTIFYBARSHOW: - conf_parse_bool (&nbar.show, val); + return conf_parse_bool (&nbar.show, val); break; case CUSTOM_CONF_NOTIFYBARDATE: (void)strncpy (nbar.datefmt, val, strlen (val) + 1); @@ -303,7 +303,7 @@ custom_set_conf (struct conf *conf, enum conf_var var, char *val) (void)strncpy (nbar.timefmt, val, strlen (val) + 1); break; case CUSTOM_CONF_NOTIFYBARWARNING: - conf_parse_int (&nbar.cntdwn, val); + return conf_parse_int (&nbar.cntdwn, val); break; case CUSTOM_CONF_NOTIFYBARCOMMAND: (void)strncpy (nbar.cmd, val, strlen (val) + 1); @@ -313,17 +313,19 @@ custom_set_conf (struct conf *conf, enum conf_var var, char *val) (void)strncpy (conf->output_datefmt, val, strlen (val) + 1); break; case CUSTOM_CONF_INPUTDATEFMT: - conf_parse_int (&conf->input_datefmt, val); + return conf_parse_int (&conf->input_datefmt, val); if (conf->input_datefmt <= 0 || conf->input_datefmt >= DATE_FORMATS) conf->input_datefmt = 1; break; case CUSTOM_CONF_DMON_ENABLE: - conf_parse_bool (&dmon.enable, val); + return conf_parse_bool (&dmon.enable, val); break; case CUSTOM_CONF_DMON_LOG: - conf_parse_bool (&dmon.log, val); + return conf_parse_bool (&dmon.log, val); break; } + + return 1; } /* Load the user configuration. */ |