<p>Joshua Colp <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8773">View Change</a></p><p>Patch set 2:</p><p>(2 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/8773/2/res/res_rtp_asterisk.c">File res/res_rtp_asterisk.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8773/2/res/res_rtp_asterisk.c@95">Patch Set #2, Line 95:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br>#define DEFAULT_RTP_BUFFER_SIZE 100<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I think 100 may be too large for now. I think 30 is more reasonable.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I was wrong on this. The browser seems to like about a 250 buffer so we need to increase it.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8773/2/res/res_rtp_asterisk.c@5267">Patch Set #2, Line 5267:</a> <code style="font-family:monospace,monospace">        for (j = 3; j < length; j++) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Oh nevermind, I see what you did.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I have figured out why this confused me and why it doesn't work.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This NACK may be part of a combined RTCP packet. As a result [3] may not point to where you expect. The current position needs to be passed in and the position used be relative to that. You also need to ensure that you don't exceed the NACK and go into another packet.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8773">change 8773</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/8773"/><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: I7f7f124af3b9d5d2fd9cffc6ba8cb48a6fff06ec </div>
<div style="display:none"> Gerrit-Change-Number: 8773 </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: Mon, 16 Apr 2018 12:28:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>