From 99013cc83a15250a6acda25f637abe9fea898266 Mon Sep 17 00:00:00 2001 From: Michal Bloch Date: Fri, 27 Apr 2018 12:41:46 +0200 Subject: [PATCH] logger: make a printing function const Log entries should never be modified by being printed so this helps prevent possible mistakes in the future. Change-Id: Ib03209db7fbcc382a73c24ababe172da5bdcdbbf Signed-off-by: Michal Bloch --- src/logger/logger.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/logger/logger.c b/src/logger/logger.c index 09529bd..2716f18 100755 --- a/src/logger/logger.c +++ b/src/logger/logger.c @@ -751,7 +751,7 @@ static int buffer_append(const struct logger_entry *entry, struct log_buffer *b) return 0; } -static int reader_print_out_single_log(struct reader *reader, struct logger_entry *logger_entry) +static int reader_print_out_single_log(struct reader *reader, const struct logger_entry *logger_entry) { assert(reader); assert(reader->buf_ptr); @@ -818,8 +818,7 @@ static int print_out_logs(struct reader *reader) } while (log_storage_reader_is_new_entry_available(reader->log_storage_reader)) { - struct logger_entry* ple = (struct logger_entry *)log_storage_reader_get_new_entry(reader->log_storage_reader); - // FIXME - ple should be const + const struct logger_entry* ple = (const struct logger_entry *)log_storage_reader_get_new_entry(reader->log_storage_reader); assert(ple); -- 2.7.4