<p>Kevin Harwell <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/11193">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bridge_softmix: use a float type to store the internal REMB bitrate<br><br>REMB's exponent is 6-bits (0..63) and has a mantissa of 18-bits. We were using<br>an unsigned integer to represent the bitrate. However, that type is not large<br>enough to hold all potential bitrate values. If the bitrate is large enough<br>bits were being shifted off the "front" of the mantissa, which caused the<br>wrong value to be sent to the browser.<br><br>This patch makes it so it now uses a float type to hold the bitrate. Using a<br>float allows for all bitrate values to be correctly represented.<br><br>ASTERISK-28255<br><br>Change-Id: Ice00fdd16693b16b41230664be5d9f0e465b239e<br>---<br>M bridges/bridge_softmix.c<br>M res/res_remb_modifier.c<br>2 files changed, 29 insertions(+), 24 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/93/11193/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/11193">change 11193</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.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.asterisk.org/c/asterisk/+/11193"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ice00fdd16693b16b41230664be5d9f0e465b239e </div>
<div style="display:none"> Gerrit-Change-Number: 11193 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>