Add some fixes that I made in regards to wideband codec handling to get
authorRussell Bryant <russell@russellbryant.com>
Thu, 20 Mar 2008 17:41:22 +0000 (17:41 +0000)
committerRussell Bryant <russell@russellbryant.com>
Thu, 20 Mar 2008 17:41:22 +0000 (17:41 +0000)
commit3c6cf5dcc5cd9359b25646002ba6c60664e2f0d0
tree7fecfcc8870ecf1fa1778dadbf0b5a542a6db154
parent3826278d38930bea52e92405622f1a531ea55163
Add some fixes that I made in regards to wideband codec handling to get
G.722 music on hold working for me.

(issue #12164, reported by milazzo and jsmith, patches by me)

res/res_musiconhold.c:
 - I moved a single line so that the sample queue update happened before
   ast_write().  The reason that this was a bug is that the G.722 frame
   originally says it has 320 samples in it (which is correct).  However,
   when the frame is written to a channel that uses RTP, main/rtp.c modifies
   the frame to cut the number of samples in half before it sends it on
   the wire.  This is to account for the stupid incorrect G.722 spec that
   makes it so we have to lie about the number of samples with RTP.  I should
   probably go and re-work the RTP code so it doesn't modify the frame so
   that a bug like this won't happen in the future.  However, this change to
   MOH is harmless.

main/channel.c:
 - I made two fixes in regards to generator timing.  Generators use samples
   for timing.  However, this code assumed 8 kHz samples.  In one case, it was
   a hard coded 160 samples, that is now written as the sample rate / 50.  The
   other place was dealing with timing a generator based on frames coming from
   the other direction.  However, that would have only worked if the sample
   rates for the formats in both directions were the same.  The code now takes
   into account that the sample rates may differ, and scales the generator
   samples accordingly.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@110268 65c4cc65-6c06-0410-ace0-fbb531ad65f3
main/channel.c
res/res_musiconhold.c