[asterisk-dev] [Code Review] 2552: Add local channel Stasis-Core messages and AMI events

[rmudgett]

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



/team/group/bridge_construction/main/core_unreal.c
<https://reviewboard.asterisk.org/r/2552/#comment17408>

    Don't put the optimize events here.  You will flood the system with events for every media frame.
    
    You have to push the calls to the callbacks deeper into ast_bridge_unreal_optimized_out() where it decides to attempt optimization and where it finds out it succeeded or not.


- rmudgett


On May 20, 2013, 2:32 a.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2552/
> -----------------------------------------------------------
> 
> (Updated May 20, 2013, 2:32 a.m.)
> 
> 
> Review request for Asterisk Developers and rmudgett.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This patch does the following:
> 
> * It adds a virtual table of callbacks to core_unreal. These callbacks can be supplied by concrete implementations of "unreal" channel drivers, which lets the unreal channel driver call specific functionality when it performs some action. Currently, this is done to notify implementations when an optimization operation has begun, and when an optimization operation has succeeded.
> 
> * It adds Stasis-Core messages for Local channel bridging and Local channel optimization. Local channel optimization is now two events: a Begin and an End. Some consumers of Stasis-Core may want to know when an operation is beginning so that they can 'prepare' their information; others will be more concerned about when the operation has completed, so that they can 'fix up' information. Stasis-Core allows for both, as does AMI.
> 
> * It also includes a memory leak fix in dial messages.
> 
> Note that the documentation included with this patch assumes that the AMI events will be turned into prefixes, as other reviews have already made this change.
> 
> 
> Diffs
> -----
> 
>   /team/group/bridge_construction/CHANGES 389195 
>   /team/group/bridge_construction/include/asterisk/core_local.h 389195 
>   /team/group/bridge_construction/include/asterisk/core_unreal.h 389195 
>   /team/group/bridge_construction/main/asterisk.c 389195 
>   /team/group/bridge_construction/main/core_local.c 389195 
>   /team/group/bridge_construction/main/core_unreal.c 389195 
>   /team/group/bridge_construction/main/manager_channels.c 389195 
>   /team/group/bridge_construction/main/stasis_channels.c 389195 
> 
> Diff: https://reviewboard.asterisk.org/r/2552/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Matt Jordan
> 
>

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