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

[Abhay Gupta]

Abhay Gupta has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/11394 )

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


Patch Set 2:

Earlier a else was added to handle AST_FRAME_NULL and such calls were marked as NOTSURE . 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 .


-- 
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: 2
Gerrit-Owner: Nasir Iqbal <nasir at ictinnovations.com>
Gerrit-Reviewer: Abhay Gupta <abhay at avissol.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Nasir Iqbal <nasir at ictinnovations.com>
Gerrit-Comment-Date: Fri, 07 Jun 2019 02:37:42 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190606/8a8be8a1/attachment.html>