<p>ipse <strong>uploaded patch set #6</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18265">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amr: Fix OA<->BWE conversion.<br><br>Size of a single AMR frame doesn't always shrink by a byte when<br>converted from octet-aligned to bandwidth-efficient mode. It does<br>shrink for AMR modes 2, 3, 4, 6, and 7 but doesn't shrink for<br>AMR modes 0, 1, 5, and SID frames because we only remove 6 bits.<br>So old code generated truncated AMR packets for those AMR modes.<br>This patch fixes the length calculation by properly counting bits.<br><br>Proper bit counting is also bringing us one small step closer to<br>properly handlig multi-frame AMR packets.<br><br>Change-Id: I0462e054a0adc9080456f3eeea9cab7c229cdb70<br>---<br>M include/osmocom/netif/amr.h<br>M src/amr.c<br>M tests/amr/amr_test.c<br>M tests/amr/amr_test.ok<br>4 files changed, 151 insertions(+), 75 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmo-netif refs/changes/65/18265/6</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18265">change 18265</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/libosmo-netif/+/18265"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-netif </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I0462e054a0adc9080456f3eeea9cab7c229cdb70 </div>
<div style="display:none"> Gerrit-Change-Number: 18265 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: ipse <Alexander.Chemeris@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>