[asterisk-dev] [Code Review] Fix problems when RTP packet frame size is changed

[Joshua Colp]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/184/#review517
-----------------------------------------------------------



/branches/1.4/main/rtp.c
<http://reviewboard.digium.com/r/184/#comment1196>

    I think it would be more efficient to just do a check on lasttxformat being non-zero before going through trying to get the current size and new size for framing and then seeing if you can muck with the smoother.


- Joshua


On 2009-03-04 20:32:40, Kevin Fleming wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/184/
> -----------------------------------------------------------
> 
> (Updated 2009-03-04 20:32:40)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> During some code analysis, I found that calling ast_rtp_codec_setpref() on an ast_rtp session does not work as expected; it does not adjust the smoother that may on the RTP session, in fact it summarily drops it, even if it has data in it, even if the current format's framing size has not changed. This is not good.
> 
> This patch changes this behavior, so that if the packetization size for the current format changes, any existing smoother is safely updated to use the new size, and if no smoother was present, one is created. A new API call for smoothers, ast_smoother_reconfigure(), was required to implement these changes.
> 
> 
> Diffs
> -----
> 
>   /branches/1.4/include/asterisk/frame.h 180297 
>   /branches/1.4/main/frame.c 180297 
>   /branches/1.4/main/rtp.c 180297 
> 
> Diff: http://reviewboard.digium.com/r/184/diff
> 
> 
> Testing
> -------
> 
> Compile testing only.
> 
> 
> Thanks,
> 
> Kevin
> 
>