<p> Attention is currently required from: Joshua Colp, Michael Kuron. </p>
<p>Patch set 8:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/19460">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File configs/samples/pjsip.conf.sample:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19460/comment/e67e2e37_8a01d5d2">Patch Set #8, Line 969:</a> <code style="font-family:monospace,monospace">                ; (default: no)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think adding a single sentence about what else actually needs to be done in order to do AOC would be helpful, e.g. "AOC updates can be sent using the AOCMessage AMI action and AOC received will emit an AMI event" or something like that.</p><p style="white-space: pre-wrap; word-wrap: break-word;">More importantly, manager.c should be updated to reflect the new level of AOC support, for both the AOCMessage AMI action and the emitted AMI events (e.g. adding AOC-S)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File doc/CHANGES-staging/res_pjsip_aoc.txt:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19460/comment/ff320121_ce6bb40a">Patch Set #8, Line 3:</a> <code style="font-family:monospace,monospace">Added res_pjsip_aoc which gives chan_pjsip the ability to send Advice-of-Charge messages</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">and receive them</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_pjsip_aoc.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19460/comment/3d19b066_8dc25cf9">Patch Set #8, Line 471:</a> <code style="font-family:monospace,monospace">                       ast_channel_name(session->channel));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You can remove these 2 lines and let the if fall through to below, since there is nothing else before the function returns.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19460/comment/d89c3582_44d49f8c">Patch Set #8, Line 478:</a> <code style="font-family:monospace,monospace"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">remove extra line of whitespace</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19460">change 19460</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/19460"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iebb7ad0d5f88526bc6629d3a1f9f11665434d333 </div>
<div style="display:none"> Gerrit-Change-Number: 19460 </div>
<div style="display:none"> Gerrit-PatchSet: 8 </div>
<div style="display:none"> Gerrit-Owner: Michael Kuron <m.kuron@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: Michael Kuron <m.kuron@gmx.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 31 Oct 2022 13:19:23 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>