[Asterisk-code-review] apps/confbridge: Added hear_own_join_sound option to control who hear... (asterisk[16])

Sean Bright asteriskteam at digium.com
Fri Apr 1 15:07:39 CDT 2022


Attention is currently required from: N A, Michael Cargile, Kevin Harwell.
Sean Bright has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/18072 )

Change subject: apps/confbridge: Added hear_own_join_sound option to control who hears sound_join
......................................................................


Patch Set 7: Code-Review-1

(2 comments)

File apps/confbridge/conf_config_parser.c:

https://gerrit.asterisk.org/c/asterisk/+/18072/comment/63d9cf2e_cce93634 
PS7, Line 101:                                         <synopsis>Determines if the user also hears the join sound when they enter a conference</synopsis>
             :                                 </configOption>
Use tabs for leading whitespace


https://gerrit.asterisk.org/c/asterisk/+/18072/comment/43869154_8d45199f 
PS7, Line 1581:                 u_profile.flags & USER_OPT_HEAR_OWN_JOIN_SOUND ?
              :                 "enabled" : "disabled");
Use tabs for leading whitespace



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/18072
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I856bd66dc0dfa057323860a6418c1371d249abd2
Gerrit-Change-Number: 18072
Gerrit-PatchSet: 7
Gerrit-Owner: Michael Cargile <mikec at vicidial.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Sean Bright <sean at seanbright.com>
Gerrit-Attention: N A <mail at interlinked.x10host.com>
Gerrit-Attention: Michael Cargile <mikec at vicidial.com>
Gerrit-Attention: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Fri, 01 Apr 2022 20:07:39 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220401/26e12ebe/attachment.html>


More information about the asterisk-code-review mailing list