<p>Joshua Colp has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/8734">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_rtp_asterisk: Queue video update on picture loss indication.<br><br>The previous payload specific feedback handling was very single<br>minded in that it just assumed everything should trigger a video<br>update. This was changed but the handling of picture loss indication<br>was not added. The result was that video may not flow. This change<br>adds it explicitly in.<br><br>Change-Id: I1894be02e39ee10a0af841b5a1dca5f0ec7d60b6<br>---<br>M include/asterisk/rtp_engine.h<br>M res/res_rtp_asterisk.c<br>2 files changed, 4 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/34/8734/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/include/asterisk/rtp_engine.h b/include/asterisk/rtp_engine.h<br>index b552948..8f044ce 100644<br>--- a/include/asterisk/rtp_engine.h<br>+++ b/include/asterisk/rtp_engine.h<br>@@ -296,6 +296,8 @@<br> #define AST_RTP_RTCP_PSFB 206<br> <br> /* Common RTCP feedback message types */<br>+/*! Picture loss indication (From RFC4585) */<br>+#define AST_RTP_RTCP_FMT_PLI 1<br> /*! Full INTRA-frame Request (From RFC5104) */<br> #define AST_RTP_RTCP_FMT_FIR 4<br> /*! REMB Information (From draft-alvestrand-rmcat-remb-03) */<br>diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c<br>index b010f6c..c87e6fb 100644<br>--- a/res/res_rtp_asterisk.c<br>+++ b/res/res_rtp_asterisk.c<br>@@ -5525,10 +5525,11 @@<br> f = &transport_rtp->f;<br> break;<br> case RTCP_PT_FUR:<br>- /* Handle RTCP FUR as FIR by setting the format to 4 */<br>+ /* Handle RTCP FUR as FIR by setting the format to 4 */<br> rc = AST_RTP_RTCP_FMT_FIR;<br> case RTCP_PT_PSFB:<br> switch (rc) {<br>+ case AST_RTP_RTCP_FMT_PLI:<br> case AST_RTP_RTCP_FMT_FIR:<br> if (rtcp_debug_test_addr(addr)) {<br> ast_verbose("Received an RTCP Fast Update Request\n");<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/8734">change 8734</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/8734"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I1894be02e39ee10a0af841b5a1dca5f0ec7d60b6 </div>
<div style="display:none"> Gerrit-Change-Number: 8734 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Joshua Colp <jcolp@digium.com> </div>