Simplify logger_init() parameters

We're passing the t_configuration_options structure anyway, no need to
pass items it contains as separate parameters.
This commit is contained in:
Ian Barwick
2015-11-19 14:03:14 +09:00
parent 3072139d06
commit f64c42a514
4 changed files with 7 additions and 6 deletions

5
log.c
View File

@@ -141,8 +141,11 @@ log_verbose(int level, const char *fmt, ...)
bool
logger_init(t_configuration_options * opts, const char *ident, const char *level, const char *facility)
logger_init(t_configuration_options * opts, const char *ident)
{
char *level = opts->loglevel;
char *facility = opts->logfacility;
int l;
int f;

3
log.h
View File

@@ -116,8 +116,7 @@ int detect_log_level(const char *level);
/* Logger initialisation and shutdown */
bool logger_init(t_configuration_options * opts, const char *ident,
const char *level, const char *facility);
bool logger_init(t_configuration_options * opts, const char *ident);
bool logger_shutdown(void);

View File

@@ -536,7 +536,7 @@ main(int argc, char **argv)
strncpy(options.loglevel, runtime_options.loglevel, MAXLEN);
}
logger_init(&options, progname(), options.loglevel, options.logfacility);
logger_init(&options, progname());
if (runtime_options.verbose)
logger_set_verbose();

View File

@@ -229,8 +229,7 @@ main(int argc, char **argv)
strerror(errno));
}
logger_init(&local_options, progname(), local_options.loglevel,
local_options.logfacility);
logger_init(&local_options, progname());
if (verbose)
logger_set_verbose();