<p> Attention is currently required from: Sean Bright, Richard Mudgett. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17784">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">Patchset:</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/+/17784?tab=comments">Patch Set #2:</a> </p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">What applications currently do this?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Things built by Asterisk users. Things that read/write channel variables (dialplan, AMI, AGI, etc). </p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Users aren't supposed to write to all caps variables. So they would use ${lastexten} not ${LASTEXTEN}. If not, that's their fault.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">As it's all caps, users shouldn't be using this variable (with this case).</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I don’t follow. Is this your opinion or is this documented somewhere?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">See Asterisk: The Definitive Guide, 5th edition, pg. 97, for example:</p><p style="white-space: pre-wrap; word-wrap: break-word;">"Note that variable names are case-sensitive. A variable named SOME DIGITS is different from a variable named SomeDigits. You should also be aware that any variables set by Asterisk will be uppercase. Some variables, such as CHANNEL and EXTEN, are reserved by Aster‐ isk. You should not attempt to set these variables"</p><p style="white-space: pre-wrap; word-wrap: break-word;">When I first started with Asterisk, I erroneously used all-caps variable names at times myself. Somebody who'd been using Asterisk since it came out quickly corrected me on that:<br>"Variable names.  Any Variable name which does not include at least ONE lower-case letter is reserved for use by the Asterisk developers.  You should never use variable names that are all upper case, because you might run into something that gets added in the future or is undocumented."</p><p style="white-space: pre-wrap; word-wrap: break-word;">So using an all-caps variable name is something we have every right to do. It is explicitly documented and intended to be that way. I don't think accounting for people doing things the wrong way changes that.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17784">change 17784</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/+/17784"/><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: Ib455fe300cc8e9a127686896ee2d0bd11e900307 </div>
<div style="display:none"> Gerrit-Change-Number: 17784 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-CC: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Attention: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 06 Jan 2022 22:59:13 +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: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>