~boiko/telephony-service/fix_voicemail_detection

Viewing all changes in revision 1059.

  • Committer: CI Train Bot
  • Author(s): You-Sheng Yang
  • Date: 2015-04-21 13:07:03 UTC
  • mfrom: (1042.1.1 telephony-service)
  • Revision ID: ci-train-bot@canonical.com-20150421130703-3lqtqvcg6q0kuni4
Class ToneGenerator creates a oneshot timer to keep track of when to send a StopTone req after successfully sending a StartEventTone one. However, when playDTMFTone() is called the second time within the timeout interval (0.2s), it's currently completely ignored and no tone will ever be generated. As a result, when someone presses dialpad buttons very quickly, some of them sound missed because not each of them is accompanied by a DTMF tone.

This patch changes the behavior as: for each playDTMFTone() call, send a StartEventTone and (re)start the timer. This way, every dialpad press causes a DTMF tone being generated and the longest duration of each tone is still 0.2 second. Fixes: #1403600
Approved by: Gustavo Pichorim Boiko

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: