<p> Attention is currently required from: N A. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/16151">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File channels/sig_analog.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/+/16151/comment/00657a96_8104b6fa">Patch Set #2, Line 1980:</a> <code style="font-family:monospace,monospace">                             pbx_builtin_setvar_helper(chan, "ANI2", anibuf + 1);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">My only comment here is I wonder if a new ANI2 channel variable should be used, as opposed to settin […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not against changing it. But I don't think that Feature Group C CAMA-MF needs its very own ANI variable presented to the user as different from a Feature Group D ANI variable. Since a user won't be using Feature Group D or C on the same channel, there's no overlap. Also, from a user perspective, if you're running FGC and FGD channels, it might be annoying that you need to remember that they use different channel variables for what is essentially the same function--to store the ANI INFO disposition--and then have to account for that in your dialplan.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/16151">change 16151</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/+/16151"/><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: Ib7e27d987aee4ed9bc3663c57ef413e21b404256 </div>
<div style="display:none"> Gerrit-Change-Number: 16151 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Sarah Autumn <sarah@endlesstemple.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </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-Comment-Date: Sat, 17 Jul 2021 15:50:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>