<p> Attention is currently required from: N A, Josh Soref. </p>
<p>Patch set 3:<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/+/17502">View Change</a></p><p>8 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File channels/chan_unistim.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/+/17502/comment/821e278f_d156af46">Patch Set #1, Line 3159:</a> <code style="font-family:monospace,monospace">                send_text_status(s, ustmtext("TransferCancel", s));</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">the corresponding localization is changed below</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'd leave this alone, it's protocol level and I think it's merely displayed text but don't know for sure.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File codecs/lpc10/analys.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/+/17502/comment/42c95fd1_dbb692fa">Patch Set #1, Line 37:</a> <code style="font-family:monospace,monospace">/* comlen control_ 12 */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Happy to drop if this is annoying</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is old code that was pulled in from the outside, provided it still actually builds I don't mind.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File include/asterisk/bridge.h:</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/+/17502/comment/afaef454_a31bd108">Patch Set #3, Line 1027:</a> <code style="font-family:monospace,monospace">void ast_bridge_set_remb_behavior(struct ast_bridge *bridge, enum ast_bridge_video_sfu_remb_behavior behavior);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This can't be changed, the old API call has to remain - although a second one can be added.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File include/asterisk/causes.h:</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/+/17502/comment/03750a03_b27d35e6">Patch Set #1, Line 100:</a> <code style="font-family:monospace,monospace">#define AST_CAUSE_MISDIALED_TRUNK_PREFIX         5</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">api?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Correct, can't be changed, has to remain.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/bridge.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/+/17502/comment/05af9d8c_f67c4590">Patch Set #3, Line 3798:</a> <code style="font-family:monospace,monospace">void ast_bridge_set_remb_behavior(struct ast_bridge *bridge, enum ast_bridge_video_sfu_remb_behavior behavior)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Per header file comment, has to remain.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/pbx.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/+/17502/comment/1a457412_26604201">Patch Set #1, Line 3947:</a> <code style="font-family:monospace,monospace">   /* Cleanup the NOTIFYs if hint is removed */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This is mostly to make my tool happy, certainly droppable. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">NOTIFY is something within SIP, but not within the core. It would be notifications.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_pjsip_publish_asterisk.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/+/17502/comment/1bda2506_2f3daece">Patch Set #1, Line 497:</a> <code style="font-family:monospace,monospace"> int cacheable = ast_json_integer_get(ast_json_object_get(json, "cacheable"));</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">this would be some sort of api change, and so either needs a compat thing or to just be dropped.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Dropping it would likely be best.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File utils/extconf.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/+/17502/comment/b7c3d464_ebcb5803">Patch Set #3, Line 3736:</a> <code style="font-family:monospace,monospace">        /* Cleanup the NOTIFYs if hint is removed */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same comment as other one regarding notifications</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17502">change 17502</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/+/17502"/><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: Ic6b76bfe3fd3f92983c5af911e91081a3a3cfa3d </div>
<div style="display:none"> Gerrit-Change-Number: 17502 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Josh Soref <jsoref@gmail.com> </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-CC: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: Josh Soref <jsoref@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 15 Dec 2021 13:26:29 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Comment-In-Reply-To: Josh Soref <jsoref@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>