Change in osmo-bsc[master]: abis_nm: Log no state change detected

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Wed Sep 30 11:52:49 UTC 2020


laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/20322 )

Change subject: abis_nm: Log no state change detected
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.osmocom.org/c/osmo-bsc/+/20322/1/src/osmo-bsc/abis_nm.c 
File src/osmo-bsc/abis_nm.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/20322/1/src/osmo-bsc/abis_nm.c@297 
PS1, Line 297: 		DEB
Why not change this to DEBUGPC(DNM, ": No state change detected\n") and move the DEBUGPC(DNM, "\n") above into the "if" clause above.  This way we always have one line about the state change, either with the suffix (no state change) or without the suffix (actual state change).



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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I15695b18ad7b31c2f9579813b76ecee36cce4ede
Gerrit-Change-Number: 20322
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge at osmocom.org>
Gerrit-Comment-Date: Wed, 30 Sep 2020 11:52:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200930/9d30a461/attachment.htm>


More information about the gerrit-log mailing list