[asterisk-dev] [Code Review] 2564: app_meetme: manager events to stasis

svnbot reviewboard at asterisk.org
Fri Jun 7 10:54:32 CDT 2013


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

(Updated June 7, 2013, 10:54 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers, kmoore and Matt Jordan.


Changes
-------

Committed in revision 390848


Bugs: ASTERISK-21467
    https://issues.asterisk.org/jira/browse/ASTERISK-21467


Repository: Asterisk


Description
-------

Refactor the events for app_meetme to go through stasis similar to confbridge.

Since there aren't any bridges involved, everything goes through channel topics (which is a little odd since some of the events don't actually involve channels, in which case they go out over the all channel topic, but since only meetme responds to the meetme message type anyway, it doesn't really matter).


Diffs
-----

  /trunk/apps/app_meetme.c 390154 
  /trunk/apps/confbridge/confbridge_manager.c 390154 
  /trunk/include/asterisk/manager.h 390154 
  /trunk/main/manager.c 390154 

Diff: https://reviewboard.asterisk.org/r/2564/diff/


Testing
-------

Checked that each event type gets properly relayed over AMI with all of the expected fields.


Thanks,

jrose

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


More information about the asterisk-dev mailing list