<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Earlier a else was added to handle AST_FRAME_NULL and such calls were marked as NOTSURE . <br>Ignoring AST_FRAME_NULL or marking NOTSURE in else is the problem, cos I think AST_FRAME_NULL can be a result of silence suppression (So must treat it as silence in AMD algorithm)><br> <br>Will that not serve the purpose instead of checking the AST_FRAME_NULL with VOICE and CNG FRAME which to me does exactly the same thing .<br>I am sorry I haven't got your point, But if we are going to take AST_FRAME_NULL as silence that it must goes with AST_FRAME_VOICE otherwise we have to repeat whole algorithm in else block.<br></pre></blockquote><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4; color: #000000;">Code-Review +1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/11394">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/11394">change 11394</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/+/11394"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-Change-Id: I16ea2d6295bc99b975e8c092e5f9fbd9214debdb </div>
<div style="display:none"> Gerrit-Change-Number: 11394 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Nasir Iqbal <nasir@ictinnovations.com> </div>
<div style="display:none"> Gerrit-Reviewer: Abhay Gupta <abhay@avissol.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: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nasir Iqbal <nasir@ictinnovations.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 07 Jun 2019 10:34:50 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>