<p>dexter <strong>uploaded patch set #7</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/10010">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">codec_pref: move match_codec_pref() to separate c-file and add unit-test<br><br>At the moment there are three sources that may advertise a list of<br>supported audio codec/rate settings. There is the MS that advertises<br>advertises a speech codec list and the MSC that sends a channel type<br>information element over A and there are also settings in the bsc<br>configuration file that may restrict the codec/rate types that are<br>allowed to use.<br><br>The function match_codec_pref() looks at all of the three buckets and<br>selects a codec that satisfies all three. This is already a somewhat<br>complicated process, overit is very isolated, so lets give it its own<br>c-file.<br><br>Due to the lack of unit-tests it is very hard to make changes here so<br>lets add also unit-test to make sure that regressions are catched early.<br><br>- Put match_codec_pref() and all its helper functions into a separate<br>  c-file.<br>- Add a unit test.<br><br>Change-Id: Iabedfdcec8b99a319f2d57cbea45c5e36c7b6e29<br>Related: OS#3361<br>---<br>M configure.ac<br>M include/osmocom/bsc/Makefile.am<br>A include/osmocom/bsc/codec_pref.h<br>M src/osmo-bsc/Makefile.am<br>A src/osmo-bsc/codec_pref.c<br>M src/osmo-bsc/osmo_bsc_bssap.c<br>M tests/Makefile.am<br>A tests/codec_pref/Makefile.am<br>A tests/codec_pref/codec_pref_test.c<br>A tests/codec_pref/codec_pref_test.ok<br>M tests/testsuite.at<br>11 files changed, 1,432 insertions(+), 160 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/10/10010/7</pre><p>To view, visit <a href="https://gerrit.osmocom.org/10010">change 10010</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/10010"/><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: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Iabedfdcec8b99a319f2d57cbea45c5e36c7b6e29 </div>
<div style="display:none"> Gerrit-Change-Number: 10010 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>