<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7392">View Change</a></p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7392/1/main/bridge_basic.c">File main/bridge_basic.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7392/1/main/bridge_basic.c@3352">Patch Set #1, Line 3352:</a> <code style="font-family:monospace,monospace">         ast_log(LOG_WARNING, "Unable to acquire target extension for attended transfer: '%s@%s'\n", exten, props->context);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">The contents of exten are undefined at this point so adding it to the message doesn't help any.  There are a couple debug messages in grab_transfer() around line 3220 that do output this information alredy when an invalid or no extension is dialed.  They are debug messages because they are output for each failed attempt by the user.  Maybe the messages should be promoted to ast_verb(3 or 4)?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7392">change 7392</a>. To unsubscribe, 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/7392"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Idac66b7628c99379cc9269158377fd87dc97a880 </div>
<div style="display:none"> Gerrit-Change-Number: 7392 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Niklas Larsson <niklas@tese.se> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 29 Nov 2017 15:02:10 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>