[asterisk-dev] [Code Review] 2648: func_channel: set dtmf_features

jrose reviewboard at asterisk.org
Mon Jul 22 10:47:44 CDT 2013

This is an automatically generated e-mail. To reply, visit:

(Updated July 22, 2013, 3:47 p.m.)

Review request for Asterisk Developers, Matt Jordan and rmudgett.


Update the review to account for Richard and Matt's findings

Bugs: ASTERISK-21876

Repository: Asterisk


Allows setting dtmf features via a channel set function (namely CHANNEL(dtmf_features)

Diffs (updated)

  /trunk/funcs/func_channel.c 395071 
  /trunk/include/asterisk/bridging_basic.h 395071 
  /trunk/main/bridging_basic.c 395071 

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


Stuck two channels in the following extension:

exten => 10,1,NoOp
exten => 10,n,Answer()
exten => 10,n,Set(CHANNEL(dtmf_features)=wktx)
exten => 10,n,Wait(50)

Then bridged them together with the manager bridge command and checked to see if both channels could park, transfer, and attended transfer.

It's worth noting that the dial application overrides this datastore entirely, so these don't get applied to dial as-is. This might need to be addressed.



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

More information about the asterisk-dev mailing list