[asterisk-dev] [Code Review] 2565: Move Hold (and now Unhold) manager event into stasis.

Jason Parker reviewboard at asterisk.org
Fri May 24 12:38:31 CDT 2013


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

(Updated May 24, 2013, 5:38 p.m.)


Review request for Asterisk Developers.


Changes
-------

Address feedback.


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


Repository: Asterisk


Description
-------

This takes the Hold event out of the few channel drivers that supported it, and moved it into stasis, so that all channel drivers use it.

This also adds a new Unhold event, removing the On/Off status from the old Hold event.


This also fixes a bug (NULL blobs didn't get sent) that would have happened with ast_queue_hangup() and possibly others, since it was affecting me here.


Diffs (updated)
-----

  /trunk/channels/chan_dahdi.c 389726 
  /trunk/channels/chan_h323.c 389726 
  /trunk/channels/chan_iax2.c 389726 
  /trunk/channels/chan_mgcp.c 389726 
  /trunk/channels/chan_misdn.c 389726 
  /trunk/channels/chan_motif.c 389726 
  /trunk/channels/chan_sip.c 389726 
  /trunk/channels/chan_skinny.c 389726 
  /trunk/channels/chan_unistim.c 389726 
  /trunk/channels/sig_analog.c 389726 
  /trunk/channels/sig_pri.c 389726 
  /trunk/include/asterisk/channel.h 389726 
  /trunk/include/asterisk/stasis_channels.h 389726 
  /trunk/main/channel.c 389726 
  /trunk/main/manager_channels.c 389726 
  /trunk/main/stasis_channels.c 389726 
  /trunk/res/res_sip_sdp_rtp.c 389726 

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


Testing
-------

Verified that chan_sip and chan_pjsip send out the new Hold and Unhold events, with and/or without musicclass.


Thanks,

Jason Parker

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


More information about the asterisk-dev mailing list