aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVadim Yanitskiy <vyanitskiy@sysmocom.de>2020-09-09 04:42:22 +0700
committerVadim Yanitskiy <vyanitskiy@sysmocom.de>2020-09-09 04:42:22 +0700
commit744236b96e08e6e775231ee08b5993a56be7f216 (patch)
tree71beae59ab5e5e44f5cf80fe2e251b46a20b011c
parent0f36421ba8dc117fdc55129f0dff54895a3d55e1 (diff)
logging: refactor and simplify log_target_destroy()
-rw-r--r--src/logging.c18
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