diff options
author | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2020-09-09 04:42:22 +0700 |
---|---|---|
committer | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2020-09-09 04:42:22 +0700 |
commit | 744236b96e08e6e775231ee08b5993a56be7f216 (patch) | |
tree | 71beae59ab5e5e44f5cf80fe2e251b46a20b011c | |
parent | 0f36421ba8dc117fdc55129f0dff54895a3d55e1 (diff) |
logging: refactor and simplify log_target_destroy()
Change-Id: Ic6b9906e6fd24de6af9c869348c100d9ef6a7df6
-rw-r--r-- | src/logging.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/src/logging.c b/src/logging.c index c14e6961..78abc199 100644 --- a/src/logging.c +++ b/src/logging.c @@ -959,16 +959,14 @@ void log_target_destroy(struct log_target *target) log_del_target(target); #if (!EMBEDDED) - if (target->output == &_file_output) { -/* since C89/C99 says stderr is a macro, we can safely do this! */ -#ifdef stderr - /* don't close stderr */ - if (target->tgt_file.out != stderr) -#endif - { - fclose(target->tgt_file.out); - target->tgt_file.out = NULL; - } + switch (target->type) { + case LOG_TGT_TYPE_FILE: + fclose(target->tgt_file.out); + target->tgt_file.out = NULL; + break; + default: + /* make GCC happy */ + break; } #endif |