[asterisk-dev] [Code Review] New Media Architecture phase 1 step 2 - Initial Translation Fixes

[David Vossel]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1064/#review3063
-----------------------------------------------------------



/trunk/main/translate.c
<https://reviewboard.asterisk.org/r/1064/#comment6304>

    Good thing I check to make sure the index is always out of bounds first :p This only affects video, which we do not even allow translation paths to be built for.  Thats why it didn't get caught during my testing.


- David


On 2010-12-21 17:15:25, David Vossel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1064/
> -----------------------------------------------------------
> 
> (Updated 2010-12-21 17:15:25)
> 
> 
> Review request for Asterisk Developers and Russell Bryant.
> 
> 
> Summary
> -------
> 
> For detailed information pertaining to this patch, please review the section of the Media Architecture document linked to below.
> 
> https://wiki.asterisk.org/wiki/display/AST/Media+Architecture+Proposal#MediaArchitectureProposal-RevisedFormatTranslation
> 
> Phase 1 Step 2 items
> -Define translation cost table.
> -Replace current matrix algorithm with new matrix algorithm using predefined costs from table.
> -Continue to use computational costs for tie breaking translators with identical src and dst formats.
> -Create table for mapping format ids to matrix index values. This is required once the conversion from the format bit field representation to a numeric value takes place and will allow for a smoother transition.
> 
> In addition to the items listed above, I have also cleaned up and reorganized the cli "core show translation ... " function.  It was difficult to maintain, and this felt like a good opportunity to clean it up.
> 
> 
> Diffs
> -----
> 
>   /trunk/include/asterisk/translate.h 299403 
>   /trunk/main/translate.c 299403 
> 
> Diff: https://reviewboard.asterisk.org/r/1064/diff
> 
> 
> Testing
> -------
> 
> Tested that translation table and paths are built correctly.
> Made some test calls requiring translation, verified paths are still created and used correctly.
> 
> 
> Thanks,
> 
> David
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20101221/1ea5dc9e/attachment-0001.htm>