<p>Joshua Colp <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6464">View Change</a></p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/6464/2/res/res_pjsip_session.c">File res/res_pjsip_session.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/6464/2/res/res_pjsip_session.c@1444">Patch Set #2, Line 1444:</a> <code style="font-family:monospace,monospace">                                         || ast_stream_get_state(stream) == AST_STREAM_STATE_REMOVED) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think this is ideal. We want to preserve ordering of streams as much as possible. This allows things to be reused and is easier on the remote client. It's only when we absolutely can't (in the case of the limit being reached) that we should remove a stream. The code in res_pjsip_sdp_rtp already handles when a stream is removed and does the right thing, and it should even work when no formats are provided. I think the correct thing here is to simply skip the compatible check further down if the state is removed.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6464">change 6464</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/6464"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ic86f8b65a4a26a60885b28b8b1a0b22e1b471d42 </div>
<div style="display:none"> Gerrit-Change-Number: 6464 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 15 Sep 2017 15:54:57 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>