<p>Stefan Engström <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6634">View Change</a></p><p>Patch set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">We change the behaviour quite a bit with patch set 1. With this change in mind, I think res = say_periodic_announcement(&qe, ringing) should be within a if (makeannouncement) block, and makeannouncement = 1 should be removed? If we do those two modifications, then the only change compared to now is we check qe.parent->announce_to_first_user at most once per Queue() execution, which is fine, but there is also the following change: <br>Before we would never call say_position in the first loop iteration, and with this change we might. I don't know whether say_position would ever do anything in the first loop iteration but it seems the whole purpose of the makeannouncement variable before was to prevent this? Actually the existing makeannouncement variable is very spurious, can we find out what its purpose was? Is the goal of this commit just a code refactoring to improve readability or performance? I'm personally already happy with the current behaviour of app_queue as of ASTERISK-27126</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6634">change 6634</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/6634"/><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: Ieaeb7dbea8ae7073086b775fbafe0625b000b10a </div>
<div style="display:none"> Gerrit-Change-Number: 6634 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Engström <stefanen@kth.se> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 29 Sep 2017 17:26:49 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>