laforge has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/libosmocore/+/15558 )
Change subject: logging_internal.h: Fix osmo_log_info definition ...................................................................... logging_internal.h: Fix osmo_log_info definition Global symbol osmo_log_info is declared in logging.c as non-const, because it is modified. As soon as logging_internal.h is included into logging.c, the compiler warns about osmo_log_info being declared twice differently. Change-Id: Iea961c3caeb12ddf60c99d4dca644bb9ab538767 --- M include/osmocom/core/logging_internal.h 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified neels: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/include/osmocom/core/logging_internal.h b/include/osmocom/core/logging_internal.h index a510f83..01c96ce 100644 --- a/include/osmocom/core/logging_internal.h +++ b/include/osmocom/core/logging_internal.h @@ -7,7 +7,7 @@ #include <osmocom/core/utils.h> extern void *tall_log_ctx; -extern const struct log_info *osmo_log_info; +extern struct log_info *osmo_log_info; extern const struct value_string loglevel_strs[]; void assert_loginfo(const char *src); -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/15558 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Iea961c3caeb12ddf60c99d4dca644bb9ab538767 Gerrit-Change-Number: 15558 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pes...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <lafo...@gnumonks.org> Gerrit-Reviewer: neels <nhofm...@sysmocom.de> Gerrit-MessageType: merged
