<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/11243">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/11243/1/src/osmo-bsc/osmo_bsc_main.c">File src/osmo-bsc/osmo_bsc_main.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/11243/1/src/osmo-bsc/osmo_bsc_main.c@885">Patch Set #1, Line 885:</a> <code style="font-family:monospace,monospace">     rc = check_codec_pref(&bsc_gsmnet->bsc_data->mscs);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think it's not only sufficient to do this here.  It should also be verified whenever any of the related input data changes, i.e. if the MSC codec preferences are changed, or if the BTS related settings change.  However, we then run into a bit of a chicken-and-egg situation on initial startup.  So those checks should be executed on any VTY change at runtime, but not during initial config file parse, as at that time not both BTS and MSC settings are already present.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This cold be a follow-up patch.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11243">change 11243</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/11243"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ice827896bab1a2330741e0fccc731a04f1a07d38 </div>
<div style="display:none"> Gerrit-Change-Number: 11243 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 06 Oct 2018 10:20:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>