<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/2c750117_f9bb3ea7">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;">I see you have it storing OLI digits in CALLERID(ANI2) now, but it's still writing to the "ANI2" cha […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I left ANI2 there on purpose. Asterisk's caller->ani2 wants an int, but I feel like ANI2 is best expressed as a string. My "ANI2" variable does that.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In cases where there are two ANI INFO digits, the first digit is usually a zero. When we 'int' it, that zero is lost, and only the nonzero digits remain. That rubs me the wrong way, since even though the nonzero digits are preserved, its still not "right", according to the book.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I specifically don't want to mess with Asterisk's feelings on what caller->ani2 should be. I do not want to add another element to the structure, since this is definitely outside the scope of what I wanted to address with this fix.</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: 3 </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: Wed, 28 Jul 2021 16:58:33 +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"> Comment-In-Reply-To: Sarah Autumn <sarah@endlesstemple.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>