[asterisk-dev] [Code Review] 2679: bridge_holding + app_bridgewait: Add more entertainment options, change default to music on hold

jrose reviewboard at asterisk.org
Wed Jul 17 10:30:37 CDT 2013

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

(Updated July 17, 2013, 3:30 p.m.)

Review request for Asterisk Developers, Matt Jordan and rmudgett.


Addresses review

Bugs: ASTERISK-21923

Repository: Asterisk


First, the arguments to app_bridgewait are changed by this patch.

Before it was just bridgewait(options)
Now it's bridgewait(role,options)

Combined with the named waiting bridge containers patch, this will soon become

The announcer option and the various idle mode options have been squashed. Now the role (announcer or participant) is specified in the role argument (defaulting to participant) while how participants act while idling in the holding bridge is set by the 'e' option which can specify an entertainment method.

New entertainment methods have been added to quell the bugbug comments. The 'e' option can now specify the following:
s - use a silence generator
h - Issue hold indications to be handled by the channel driver (respects MOH class argument)
n - provide no entertainment (used to be the default)

in addition to the existing:
m - Play music on hold directly (default) (respects MOH class argument)
r - Ring ring

Hopefully these changes make the app more self-explanatory to use since it isn't possible to use conflicting options now.

Between this review and https://reviewboard.asterisk.org/r/2642/diff/#index_header all BUGBUG comments in app_bridgewait should be quelled.

Diffs (updated)

  /trunk/apps/app_bridgewait.c 394249 
  /trunk/bridges/bridge_holding.c 394249 
  /trunk/main/bridging_roles.c 394249 

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


* Tested all new entertainment methods to make sure they produced the anticipated audio (or just RTP in the case of silence and nothing in the case of nothing)
* Tested all options to BridgeWait to make sure they didn't break
* Tested to make sure roles were applied properly and that unknown roles would be appropriately rejected.



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

More information about the asterisk-dev mailing list