Attention is currently required from: laforge, pespin, dexter.
osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/26950 )

Change subject: logging: log to stdout when logging is not initialized
......................................................................


Patch Set 1:

(1 comment)

Patchset:

PS1:
I don't understand the use case, isn't this a bug in the program using 
libosmocore, if it attempts to log before initializing logging?

With this patch it seems that it won't complain at all about logging not being 
initialized, since the log_check_level -> assert_loginfo() code path isn't 
executed. Is this intended?



--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/26950
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I9b1b0988e02322e3e44fd4ceea3e1bc2d4df3c45
Gerrit-Change-Number: 26950
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Wed, 19 Jan 2022 13:36:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to