Enable wide band audio in musiconhold streams.
authorWalter Doekes <walter+asterisk@wjd.nu>
Fri, 17 Jan 2014 14:17:04 +0000 (14:17 +0000)
committerWalter Doekes <walter+asterisk@wjd.nu>
Fri, 17 Jan 2014 14:17:04 +0000 (14:17 +0000)
Review: https://reviewboard.asterisk.org/r/3112/

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

CHANGES
res/res_musiconhold.c

diff --git a/CHANGES b/CHANGES
index 6601452..5e0cb47 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -45,6 +45,11 @@ Directory
      USEREXIT    user pressed '#' from the selection prompt to exit
      FAILED      directory failed in a way that wasn't accounted for. Dang.
 
+MusicOnHold
+--------------------------
+ * MusicOnHold streams (all modes other than "files") now support wide band
+   audio too.
+
 Page
 --------------------------
  * Added options 'b' and 'B' to apply predial handlers for outgoing calls
index 086139a..58c908d 100644 (file)
@@ -704,7 +704,8 @@ static void *monmp3thread(void *data)
                                        ast_log(LOG_ERROR, "Failed to acknowledge timer for mp3player\n");
                                        return NULL;
                                }
-                               res = 320;
+                               /* 25 samples per second => 40ms framerate => 320 samples */
+                               res = 320; /* 320/40 = 8 samples/ms */
                        } else {
                                ast_log(LOG_WARNING, "poll() failed: %s\n", strerror(errno));
                                res = 0;
@@ -725,8 +726,12 @@ static void *monmp3thread(void *data)
                                ast_log(LOG_NOTICE, "Request to schedule in the past?!?!\n");
                                deadline = tv_tmp;
                        }
-                       res = 8 * MOH_MS_INTERVAL;      /* 8 samples per millisecond */
+                       /* 10 samples per second (MOH_MS_INTERVAL) => 100ms framerate => 800 samples */
+                       res = 8 * MOH_MS_INTERVAL; /* 800/100 = 8 samples/ms */
                }
+               /* For non-8000Hz formats, we need to alter the resolution */
+               res = res * ast_format_rate(&class->format) / 8000;
+
                if ((strncasecmp(class->dir, "http://", 7) && strcasecmp(class->dir, "nodir")) && AST_LIST_EMPTY(&class->members))
                        continue;
                /* Read mp3 audio */