[asterisk-dev] [Code Review] 3649: Bridging: Allow channels to define additional bridging hooks

opticron reviewboard at asterisk.org
Wed Jun 25 07:54:33 CDT 2014


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

(Updated June 25, 2014, 7:54 a.m.)


Review request for Asterisk Developers and rmudgett.


Changes
-------

Address Mark's and Richard's comments.


Bugs: AFS-78
    https://issues.asterisk.org/jira/browse/AFS-78


Repository: Asterisk


Description
-------

This patch allows the current owner of a channel to define various feature hooks to be made available once the channel has entered a bridge.

Currently, it is only possible to attach these hooks by calling ast_bridge_join or as_bridge_impart directly.


Diffs (updated)
-----

  trunk/tests/test_channel_datastore_features.c PRE-CREATION 
  trunk/main/channel.c 416731 
  trunk/main/bridge_channel.c 416731 
  trunk/main/bridge.c 416736 
  trunk/include/asterisk/channel.h 416731 
  trunk/include/asterisk/bridge_features.h 416731 

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


Testing
-------

Ensured that DTMF hooks could be attached to the channel via this mechanism and that they became available and were executable once the channel entered a bridge via Dial().


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140625/a77b062f/attachment.html>


More information about the asterisk-dev mailing list