]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Let channels joining a MeetMe conference opt out of the denoiser
authorMatthew Jordan <mjordan@digium.com>
Thu, 28 Feb 2013 16:49:46 +0000 (16:49 +0000)
committerMatthew Jordan <mjordan@digium.com>
Thu, 28 Feb 2013 16:49:46 +0000 (16:49 +0000)
commit7191cb4be62eb527ae799964114a0b8b896ace47
tree15e38d5332c849f98e394e0fa2f146da90bdb219
parented28f2194f606ed9f8f3fc4f43fa2950c928b2aa
Let channels joining a MeetMe conference opt out of the denoiser

For some channel drivers, specifically those that have a varying rate in the
number of audio samples, the audio quality for a MeetMe conference can be
exceedingly poor. This is due to a unilateral application of the DENOISE
function in func_speex to channels joining the conference.

The denoiser function in the speex library is initialized with the number of
audio samples in each sample that will be provided to it. If the number of
audio samples changes, the denoiser has to be thrown away and re-initialized.

While this could be worked around by removing func_speex, that doesn't help
if you actually use the denoiser with other channels on the system.

This patches does the following:
 * Checks for the presence of func_speex as opposed to codec_speex when
   determining if the DENOISE function is present (which is where the function
   is actually implemented)
 * Adds an option to MeetMe 'n' that causes the denoiser to not be applied
   to a channel when it joins. This keeps the current behavior the default, but
   let's users disable the denoiser if it causes problems on their system.

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

(closes issue AST-1062)
Reported by: Thomas Arimont
........

Merged revisions 382227 from http://svn.asterisk.org/svn/asterisk/branches/1.8

git-svn-id: https://origsvn.digium.com/svn/asterisk/certified/branches/1.8.15@382228 65c4cc65-6c06-0410-ace0-fbb531ad65f3
UPGRADE.txt
apps/app_meetme.c