]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Merged revisions 163448 via svnmerge from
authorRussell Bryant <russell@russellbryant.com>
Fri, 12 Dec 2008 13:55:30 +0000 (13:55 +0000)
committerRussell Bryant <russell@russellbryant.com>
Fri, 12 Dec 2008 13:55:30 +0000 (13:55 +0000)
commit7fcac067b2efca15bfd6e2ed8aec3d16165f23c3
tree143f7c7d4409292fc6d174119b3557cfc70875ef
parent592cab82026262bec92a6dec80a4d53cb13d1aa1
Merged revisions 163448 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r163448 | russell | 2008-12-12 07:44:08 -0600 (Fri, 12 Dec 2008) | 26 lines

Resolve issues that could cause DTMF to be processed out of order.

These changes come from team/russell/issue_12658

1) Change autoservice to put digits on the head of the channel's frame readq
   instead of the tail.  If there were frames on the readq that autoservice
   had not yet read, the previous code would have resulted in out of order
   processing.  This required a new API call to queue a frame to the head
   of the queue instead of the tail.

2) Change up the processing of DTMF in ast_read().  Some of the problems
   were the result of having two sources of pending DTMF frames.  There
   was the dtmfq and the more generic readq.  Both were used for pending
   DTMF in various scenarios.  Simplifying things to only use the frame
   readq avoids some of the problems.

3) Fix a bug where a DTMF END frame could get passed through when it
   shouldn't have.  If code set END_DTMF_ONLY in the middle of digit emulation,
   and a digit arrived before emulation was complete, digits would get
   processed out of order.

(closes issue #12658)
Reported by: dimas
Tested by: russell, file
Review: http://reviewboard.digium.com/r/85/

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@163449 65c4cc65-6c06-0410-ace0-fbb531ad65f3
include/asterisk/channel.h
main/autoservice.c
main/channel.c