[Asterisk-code-review] Revert "RTP: reset DTMF last seqno/timestamp on voice packet... (asterisk[13.24])

George Joseph asteriskteam at digium.com
Tue Dec 11 14:21:40 CST 2018


George Joseph has submitted this change and it was merged. ( https://gerrit.asterisk.org/10791 )

Change subject: Revert "RTP: reset DTMF last seqno/timestamp on voice packet with marker bit"
......................................................................

Revert "RTP: reset DTMF last seqno/timestamp on voice packet with marker bit"

This reverts commit cf620ce0f6dc4a7ef713dc85b4c2610b614cd647.

Pending resolution of ASTERISK_28200

Change-Id: If18ab1166db0d634ce8a099bd7460251e9c682e1
---
M res/res_rtp_asterisk.c
1 file changed, 0 insertions(+), 10 deletions(-)

Approvals:
  George Joseph: Looks good to me, approved; Approved for Submit



diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c
index 4586ca6..8841f6d 100644
--- a/res/res_rtp_asterisk.c
+++ b/res/res_rtp_asterisk.c
@@ -5732,16 +5732,6 @@
 	switch (ast_format_get_type(rtp->f.subclass.format)) {
 	case AST_MEDIA_TYPE_AUDIO:
 		rtp->f.frametype = AST_FRAME_VOICE;
-
-		/* The marker bit set on the voice packet indicates the start
-		 * of a new stream and a new time stamp. Need to reset the DTMF
-		 * last sequence number and the timestamp of the last END packet.
-		 */
-		if (mark) {
-			rtp->last_seqno = 0;
-			rtp->last_end_timestamp = 0;
-		}
-
 		break;
 	case AST_MEDIA_TYPE_VIDEO:
 		rtp->f.frametype = AST_FRAME_VIDEO;

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

Gerrit-Project: asterisk
Gerrit-Branch: 13.24
Gerrit-MessageType: merged
Gerrit-Change-Id: If18ab1166db0d634ce8a099bd7460251e9c682e1
Gerrit-Change-Number: 10791
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation (1000185)
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181211/cedf1284/attachment.html>


More information about the asterisk-code-review mailing list