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

opticron reviewboard at asterisk.org
Wed May 29 14:58:20 CDT 2013


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

Ship it!


Other than the documentation nitpick below, this looks ready to go in.


/team/group/bridge_construction/CHANGES
<https://reviewboard.asterisk.org/r/2552/#comment17086>

    effects -> affects


- opticron


On May 19, 2013, 9:32 p.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2552/
> -----------------------------------------------------------
> 
> (Updated May 19, 2013, 9:32 p.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/20130529/aa15d0d4/attachment.htm>


More information about the asterisk-dev mailing list