<p style="white-space: pre-wrap; word-wrap: break-word;">This patch is not necessary.</p><p style="white-space: pre-wrap; word-wrap: break-word;">When you get a connected line update, the connected line information is already set to the new connected party (Bob) because you are no longer connected to the old party (Alice).  There is no relationship between the old and the new information because Bob is not Alice.  Thus in the normal case this patch wastes resources setting up channel variables and allocating memory that is not needed.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The only reason to revert the connected line information is if you wanted it to stay constant.  There are already several ways to get the constant information to reset the connected line information.  These ways do not require any changes to Asterisk and incur the storage overhead only when this behavior is needed.  I already provided example dialplan in this other review https://gerrit.asterisk.org/#/c/asterisk/+/10286/ to do just that.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -2</span></p><p><a href="https://gerrit.asterisk.org/10299">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10299">change 10299</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/10299"/><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: Ie39f7a3402101333531884873cb353c73cd5334d </div>
<div style="display:none"> Gerrit-Change-Number: 10299 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexei Gradinari <alex2grad@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 (1000185) </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 28 Sep 2018 17:54:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>