Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/37216?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: logging: use ?LOG macros instead of calling logger:level()
......................................................................
logging: use ?LOG macros instead of calling logger:level()
The difference between using these macros and the exported functions
is that macros add location (originator) information to the metadata,
and perform lazy evaluation by wrapping the logger call in a case
statement, so it is only evaluated if the log level of the event
passes the primary log level check.
https://www.erlang.org/doc/apps/kernel/logger_chapter.html#logger-api
Change-Id: I3b6e4296f736f9c87d2706ef5fb75146fd9f1239
---
M src/s1ap_proxy.erl
M src/sctp_client.erl
M src/sctp_proxy.erl
M src/sctp_server.erl
4 files changed, 63 insertions(+), 41 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-s1gw
refs/changes/16/37216/2
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/37216?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I3b6e4296f736f9c87d2706ef5fb75146fd9f1239
Gerrit-Change-Number: 37216
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
