<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20322">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20322/1/src/osmo-bsc/abis_nm.c">File src/osmo-bsc/abis_nm.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20322/1/src/osmo-bsc/abis_nm.c@297">Patch Set #1, Line 297:</a> <code style="font-family:monospace,monospace">          DEB</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">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).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20322">change 20322</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bsc/+/20322"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I15695b18ad7b31c2f9579813b76ecee36cce4ede </div>
<div style="display:none"> Gerrit-Change-Number: 20322 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 30 Sep 2020 11:52:49 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>