[asterisk-dev] [Code Review] 3008: app_confbridge: Add an option 'announce_join_leave_review' for users that adds recording review to the 'announce_join_leave' name recording feature

Jonathan Rose reviewboard at asterisk.org
Mon Nov 11 14:08:33 CST 2013

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

Review request for Asterisk Developers, Mark Michelson and rmudgett.

Repository: Asterisk


Pretty straight forward really.  Adds the same basic functionality provided by the 'i' option to MeetMe into Confbridge (which already implemented functionality similar to the 'I' option through announce_join_leave)


  /trunk/configs/confbridge.conf.sample 402603 
  /trunk/apps/confbridge/include/confbridge.h 402603 
  /trunk/apps/confbridge/conf_config_parser.c 402603 
  /trunk/apps/app_confbridge.c 402603 
  /trunk/CHANGES 402603 

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


Made sure the recording feature worked with and without the review option in the expected way (with and without recording review). It's a pretty straight forward patch though.


Jonathan Rose

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

More information about the asterisk-dev mailing list