<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/15373">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15373/2/doc/manuals/chapters/smscb.adoc">File doc/manuals/chapters/smscb.adoc:</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/15373/2/doc/manuals/chapters/smscb.adoc@43">Patch Set #2, Line 43:</a> <code style="font-family:monospace,monospace">configuration more simple, ti also can operate in TCP client mode,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">typo: ti->it.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15373/2/doc/manuals/chapters/smscb.adoc@44">Patch Set #2, Line 44:</a> <code style="font-family:monospace,monospace">connecting to the CBC.  This way the BSCs all need to know the CBC IP</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">1- Extra space after dot.<br>2- "This way the BSCs all need to know IS the CBC IP address".</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15373/2/doc/manuals/chapters/smscb.adoc@45">Patch Set #2, Line 45:</a> <code style="font-family:monospace,monospace">address, but not vice-versa</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Missing dot.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15373/2/src/osmo-bsc/smscb.c">File src/osmo-bsc/smscb.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/15373/2/src/osmo-bsc/smscb.c@44">Patch Set #2, Line 44:</a> <code style="font-family:monospace,monospace">/* replace the old head of an entire list with e new head; effectively moves the entire</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"a new head"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15373/2/src/osmo-bsc/smscb.c@142">Patch Set #2, Line 142:</a> <code style="font-family:monospace,monospace">    static struct osmo_cell_global_id cgi;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">What about passing it as a parameter and filling it from there? This way space is not lost for users who don't plan to use CBCH.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/15373">change 15373</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/+/15373"/><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: Ia0a0de862a104d0f447a5d6e56c7c83981b825c7 </div>
<div style="display:none"> Gerrit-Change-Number: 15373 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 02 Sep 2019 09:41:21 +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>