<p> Attention is currently required from: Sean Bright. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/16238">View Change</a></p><p>2 comments:</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/+/16238?tab=comments">Patch Set #1:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">> The documentation for this application says that it plays a continuous 1004MHz tone. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ack</p></li></ul></li><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/+/16238?tab=comments">Patch Set #3:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The default should be the current behavior. The optional argument would enable the 1 second pause.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Even though the default behavior is incorrect?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Nobody seriously using this application wants the existing behavior. It's not how a Milliwatt test line works (probably whoever wrote this was unfamiliar with the specification). Shouldn't the default be "correct" behavior, with an option to enable "incorrect" behavior? Why keep an invalid default when we know it's wrong?</p><p style="white-space: pre-wrap; word-wrap: break-word;">If you call any real Milliwatt number (e.g. (503)-697-1000), they all have the 1s pause and people expect that an application called "Milliwatt" will be... an actual Milliwatt.</p><p style="white-space: pre-wrap; word-wrap: break-word;">e.g. pg. 9 here: https://downloads.avaya.com/elmodocs2/comm_mgr/r5_1/03-603118_AnalogTestHeal_June2008.pdf</p><p style="white-space: pre-wrap; word-wrap: break-word;">The official AT&T specification here: https://primeaccess.att.com//commonfiles/Swbt/1482/Description%20of%20Test%20Lines.doc</p><p style="white-space: pre-wrap; word-wrap: break-word;">The existing default isn't a Milliwatt, it's just a loud PlayTones(1004).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/16238">change 16238</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/+/16238"/><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: I73ccc6c6fcaa31931c6fff3b85ad1805b2ce9d8c </div>
<div style="display:none"> Gerrit-Change-Number: 16238 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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-Attention: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 13 Aug 2021 13:52:28 +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"> Gerrit-MessageType: comment </div>