<p style="white-space: pre-wrap; word-wrap: break-word;">Tests needs update</p><p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/17997">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/17997/1/include/osmocom/netif/amr.h">File include/osmocom/netif/amr.h:</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/libosmo-netif/+/17997/1/include/osmocom/netif/amr.h@96">Patch Set #1, Line 96:</a> <code style="font-family:monospace,monospace">#define AMR_FT_0_LEN_BITS       95      /* 4.75 */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I lack some documentation here like for the above. IIUC these are the same as the above ones but in bits. So it probably makes sense to redefine the above ones for instance:<br>#define AMR_FT_0_LEN ((AMR_FT_0_LEN_BITS + 7) / 8)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/17997/1/src/amr.c">File src/amr.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/libosmo-netif/+/17997/1/src/amr.c@44">Patch Set #1, Line 44:</a> <code style="font-family:monospace,monospace">static size_t amr_ft_to_bytes[AMR_FT_MAX] = {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Since we now have it in bits, we can probably drop this static array and use (amr_ft_to_bits[ft] + 7)/8.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-netif/+/17997">change 17997</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/+/17997"/><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: I9fc5fb92e9bada22a47a82fcfb0925e892e50ced </div>
<div style="display:none"> Gerrit-Change-Number: 17997 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 02 May 2020 16:21:50 +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>