[asterisk-dev] audiohook with G722 codec: wrong number of samples

[Simon Sester]

The issue is now on the bugtracker:
ID: 17267 https://issues.asterisk.org/view.php?id=17267

My fix is attached to the report.

I hope someone who has more experience with developing asterisk can look
at the issue and find a complete solution.

Simon


On Thu, 2010-04-29 at 15:26 +0200, Klaus Darilion wrote:
> Hi Simon!
> 
> Thanks for the patch. But you have to create an issue on the bugtracker 
> for this patch and accept the disclaimer - otherwise the patch can not 
> be accepted.
> https://issues.asterisk.org/
> 
> regards
> klaus
> 
> Am 29.04.2010 14:19, schrieb Simon Sester:
> > On Wed, 2010-04-28 at 11:06 -0500, David Vossel wrote:
> >> Simon,
> >>
> >> If you are using audiohooks, everything is downgraded to 8khz 16bit signed linear, so if we have 16khz G722 with 320 samples, that will be downgraded to 160 sample 8khz audio for the audiohook manipulation.  This is a limitation that I am not happy with.
> >>
> >> David Vossel
> >> Digium, Inc. | Software Developer, Open Source Software
> >> 445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
> >> Check us out at: www.digium.com&  www.asterisk.org
> >> The_Boy_Wonder in #asterisk-dev
> >
> > David,
> >
> > Thanks for your help.
> >
> > For my application I need all 320 samples so I made some changes to the
> > audiohook.c so that is passes 16khz 16bit signed linear
> > (AST_FORMAT_SLINEAR16) to the audiohook callback function if necessary.
> > I attached a diff to my mail. The hack works fine for me, but as far as
> > I can tell from looking at the code it only works for TYPE-MANIPULATE
> > audiohooks.
> >
> > Simon
> >
> >