Merged revisions 257686 via svnmerge from
authorDwayne M. Hubbard <dwayne.hubbard@gmail.com>
Fri, 16 Apr 2010 21:22:30 +0000 (21:22 +0000)
committerDwayne M. Hubbard <dwayne.hubbard@gmail.com>
Fri, 16 Apr 2010 21:22:30 +0000 (21:22 +0000)
commit77868073a81c39859957d28f4e66104756dd2f46
treea79f2d26f2392a2a39de569c662fbd9fd0b81945
parentba81ee6d2867bc7f6702d15e597ec567ef3b34e2
Merged revisions 257686 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r257686 | dhubbard | 2010-04-16 16:15:43 -0500 (Fri, 16 Apr 2010) | 21 lines

  Make the mixmonitor thread process audio frames faster

  Mantis issue 17078 reports MixMonitor recordings have shorter durations than
  the call duration.  This was because the mixmonitor thread was not processing
  frames from the audiohook fast enough.  The mixmonitor thread would slowly fall
  behind the most recent audio frame and when the channel hangs up, the mixmonitor
  thread would exit without processing the same number of frames as the channel;
  leaving the mixmonitor recording shorter than actual call duration.

  This revision fixes this issue by moving the ast_audiohook_trigger_wait() and
  the subsequent audiohook.status check into the block where the
  ast_audiohook_read_frame() function returns NULL.

  (closes issue #17078)
  Reported by: geoff2010
  Patches:
        dw-M17078.patch uploaded by dhubbard (license 733)
  Tested by: dhubbard, geoff2010

  Review: https://reviewboard.asterisk.org/r/611/
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@257713 65c4cc65-6c06-0410-ace0-fbb531ad65f3
apps/app_mixmonitor.c