<p>Joshua Colp <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/6032">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bridge_softmix / res_rtp_asterisk: Fix packet loss and renegotiation issues.<br><br>This change does a few things to improve packet loss and renegotiation:<br><br>1. On outgoing RTP streams we will now properly reflect out of order<br>packets and packet loss in the sequence number. This allows the<br>remote jitterbuffer to better reorder things.<br><br>2. Video updates can now be discarded for a period of time<br>after one has been sent to prevent flooding of clients.<br><br>3. For declined and removed streams we will now release any<br>media session resources associated with them. This was not<br>previously done and caused an issue where old state was being<br>used for a new stream.<br><br>4. RTP bundling was not actually removing bundled RTP instances<br>from the parent. This has been resolved by removing based on<br>the RTP instance itself and not the SSRC.<br><br>5. The code did not properly handle explicitly unbundling an<br>RTP instance from its parent. This now works as expected.<br><br>ASTERISK-27143<br><br>Change-Id: Ibd91362f0e4990b6129638e712bc8adf0899fd45<br>---<br>M apps/app_confbridge.c<br>M apps/confbridge/conf_config_parser.c<br>M apps/confbridge/include/confbridge.h<br>M bridges/bridge_softmix.c<br>M bridges/bridge_softmix/include/bridge_softmix_internal.h<br>M configs/samples/confbridge.conf.sample<br>M include/asterisk/bridge.h<br>M include/asterisk/frame.h<br>M main/bridge.c<br>M main/rtp_engine.c<br>M res/res_pjsip_session.c<br>M res/res_rtp_asterisk.c<br>12 files changed, 108 insertions(+), 9 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/32/6032/4</pre><p>To view, visit <a href="https://gerrit.asterisk.org/6032">change 6032</a>. To unsubscribe, 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/6032"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Ibd91362f0e4990b6129638e712bc8adf0899fd45 </div>
<div style="display:none"> Gerrit-Change-Number: 6032 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>