[Asterisk-code-review] app_amd: issue with silence suppression fixed (...asterisk[13])

Nasir Iqbal asteriskteam at digium.com
Tue May 21 01:48:53 CDT 2019


Nasir Iqbal has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/11394


Change subject: app_amd: issue with silence suppression fixed
......................................................................

app_amd: issue with silence suppression fixed

Now AMD algorithm will not ignore AST_FRAME_NULL, As I thing using manual
wait time instead of `framelength` is enough to fix timeout / TOOLONG issue.

ASTERISK-28419 #close
Change-Id: I3616aa47e59d0371d7d610c49037b0a619c155d8

Change-Id: I16ea2d6295bc99b975e8c092e5f9fbd9214debdb
---
M apps/app_amd.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/94/11394/1

diff --git a/apps/app_amd.c b/apps/app_amd.c
index f56fe79..1c43591 100644
--- a/apps/app_amd.c
+++ b/apps/app_amd.c
@@ -295,7 +295,7 @@
 			break;
 		}
 
-		if (f->frametype == AST_FRAME_VOICE || f->frametype == AST_FRAME_CNG) {
+		if (f->frametype == AST_FRAME_VOICE || f->frametype == AST_FRAME_NULL || f->frametype == AST_FRAME_CNG) {
 			/* Figure out how long the frame is in milliseconds */
 			if (f->frametype == AST_FRAME_VOICE) {
 				framelength = (ast_codec_samples_count(f) / DEFAULT_SAMPLES_PER_MS);

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/11394
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: I16ea2d6295bc99b975e8c092e5f9fbd9214debdb
Gerrit-Change-Number: 11394
Gerrit-PatchSet: 1
Gerrit-Owner: Nasir Iqbal <nasir at ictinnovations.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190521/5a5509f0/attachment.html>


More information about the asterisk-code-review mailing list