From 1bc478656edb7d25a2765fb698c26f612e83f085 Mon Sep 17 00:00:00 2001 From: Paul Belanger Date: Mon, 7 Jun 2010 17:34:45 +0000 Subject: Set threshold for silence detection defaults to 256 (closes issue #15685) Reported by: david_s5 Patches: dsp-silence-threshold-init.diff uploaded by dant (license 670) issue15685.patch.v5 uploaded by pabelanger (license 224) Tested by: danti Review: https://reviewboard.asterisk.org/r/670/ git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@268690 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- main/dsp.c | 22 +++++++++++++++++----- 1 file changed, 17 insertions(+), 5 deletions(-) (limited to 'main/dsp.c') diff --git a/main/dsp.c b/main/dsp.c index 649ae9ffd..cba01b5f1 100644 --- a/main/dsp.c +++ b/main/dsp.c @@ -213,6 +213,12 @@ enum gsamp_thresh { /* How many successive misses needed to consider end of a digit */ #define DTMF_MISSES_TO_END 3 +/*! + * \brief The default silence threshold we will use if an alternate + * configured value is not present or is invalid. + */ +static const int DEFAULT_SILENCE_THRESHOLD = 256; + #define CONFIG_FILE_NAME "dsp.conf" typedef struct { @@ -1674,19 +1680,25 @@ static int _dsp_init(int reload) struct ast_config *cfg; cfg = ast_config_load2(CONFIG_FILE_NAME, "dsp", config_flags); - if (cfg == CONFIG_STATUS_FILEMISSING || cfg == CONFIG_STATUS_FILEUNCHANGED || cfg == CONFIG_STATUS_FILEINVALID) { + if (cfg == CONFIG_STATUS_FILEMISSING || cfg == CONFIG_STATUS_FILEINVALID) { + ast_verb(5, "Can't find dsp config file %s. Assuming default silencethreshold of %d.\n", CONFIG_FILE_NAME, DEFAULT_SILENCE_THRESHOLD); + thresholds[THRESHOLD_SILENCE] = DEFAULT_SILENCE_THRESHOLD; + return 0; + } + + if (cfg == CONFIG_STATUS_FILEUNCHANGED) { return 0; } - if (cfg && cfg != CONFIG_STATUS_FILEUNCHANGED) { + if (cfg) { const char *value; value = ast_variable_retrieve(cfg, "default", "silencethreshold"); if (value && sscanf(value, "%30d", &thresholds[THRESHOLD_SILENCE]) != 1) { - ast_log(LOG_WARNING, "%s: '%s' is not a valid silencethreshold value\n", CONFIG_FILE_NAME, value); - thresholds[THRESHOLD_SILENCE] = 256; + ast_verb(5, "%s: '%s' is not a valid silencethreshold value\n", CONFIG_FILE_NAME, value); + thresholds[THRESHOLD_SILENCE] = DEFAULT_SILENCE_THRESHOLD; } else if (!value) { - thresholds[THRESHOLD_SILENCE] = 256; + thresholds[THRESHOLD_SILENCE] = DEFAULT_SILENCE_THRESHOLD; } ast_config_destroy(cfg); -- cgit v1.2.3