[asterisk-bugs] [Asterisk 0016230]: [patch] g726 to g726aal2 translation and cost-calculation are wrong but easily fixed.

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16230 
====================================================================== 
Reported By:                jonconsi
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   16230
Category:                   Codecs/codec_g726
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     confirmed
Asterisk Version:           SVN 
JIRA:                       SWP-787 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.1 
SVN Revision (number only!): 229607 
Request Review:              
====================================================================== 
Date Submitted:             2009-11-12 07:09 CST
Last Modified:              2010-02-02 11:56 CST
====================================================================== 
Summary:                    [patch] g726 to g726aal2 translation and
cost-calculation are wrong but easily fixed.
Description: 
As detailed in Issue 15504, sometimes Asterisk incorrectly calculates that
it is lower-cost to go from g726->g726AAL2->linear instead of directly from
g726->linear. This exposed a bug in the g726->g726AAL2 (or vice-versa)
translation, which is a simple nybble re-ordering. This bug gives rise to
badly distorted and choppy audio.

The fix given in 15504 was to delete the g726->g726AAL2 translation but
this is unneccessary.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0015504 [patch] G726 Codec has choppy audio on ...
====================================================================== 

---------------------------------------------------------------------- 
 (0117539) qwell (administrator) - 2010-02-02 11:56
 https://issues.asterisk.org/view.php?id=16230#c117539 
---------------------------------------------------------------------- 
The XtoY_samples method has been intentionally changed, since it is not
required.  You should continue using g726_samples, and slin8_samples
(rather than g726toslin_samples and slintog726_samples).  If they need to
be corrected somehow, that's fine, but please do not re-add those
structures. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-02-02 11:56 qwell          Note Added: 0117539                          
======================================================================