[Asterisk-code-review] Changed app amd for ASTERISK-28143 (asterisk[16])

Joshua Colp asteriskteam at digium.com
Mon Nov 26 08:34:46 CST 2018


Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/10707 )

Change subject: Changed app_amd for ASTERISK-28143
......................................................................


Patch Set 1:

The previous change explicitly removed NULL frame handling for a reason to fix a bug that was experienced. You're now adding it back in to fix a bug you've seen. We can't keep removing it and adding it back in, so understanding both cases is needed to determine the proper fix.


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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia4fe35be6bd9683bc1ae01b884d7a544c128d3b6
Gerrit-Change-Number: 10707
Gerrit-PatchSet: 1
Gerrit-Owner: Abhay Gupta <abhay at avissol.com>
Gerrit-Reviewer: Abhay Gupta <abhay at avissol.com>
Gerrit-Reviewer: Jenkins2 (1000185)
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Comment-Date: Mon, 26 Nov 2018 14:34:46 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181126/a2ea740b/attachment-0001.html>


More information about the asterisk-code-review mailing list