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

Matt Jordan reviewboard at asterisk.org
Tue Jul 2 14:21:50 CDT 2013


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

(Updated July 2, 2013, 7:21 p.m.)


Review request for Asterisk Developers and rmudgett.


Changes
-------

Addressed Richard's and Kinsey's findings.


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 (updated)
-----

  /trunk/CHANGES 393396 
  /trunk/include/asterisk/bridging.h 393396 
  /trunk/include/asterisk/core_local.h 393396 
  /trunk/include/asterisk/core_unreal.h 393396 
  /trunk/main/bridging.c 393396 
  /trunk/main/core_local.c 393396 
  /trunk/main/core_unreal.c 393396 
  /trunk/main/manager_channels.c 393396 

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/20130702/4a578c86/attachment.htm>


More information about the asterisk-dev mailing list