<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;">Aha, thanks for clearing a few things up. Good point  noticing that announce-to-first-user is off by default. I did not consider that when guessing the impact of ASTERISK-27216. I based the changes of ASTERISK-27216 off what I thought was the most logical interpretation of the variable announce-to-first-user and at the time I believed the default value was 1 (we have always had it on explicitly). My interpretation seems to be more in line with what the description for the parameter is in ./configs/samples/queues.conf.sample (...If enabled, play announcements to the first user waiting in the Queue...). Although that does not explicidly say "If and only if enabled", it's sort of implied... and I interpret announcements as both position and periodic announcements. If one instead uses how asterisk has actually worked for a long time as a source of truth for expected behaviour, I think I understand your perspective better. If we would go towards your line play; I think renaming variables to reflect their purpose and totally rewriting the parameter description in queues.conf would be in order. Going towards my line of play would definitely need to address the fact most asterisk users probably have that parameter set to 0 and still expect their head callers to hear periodic info once in a while due to historic reasons...</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: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Engström <stefanen@kth.se> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 02 Oct 2017 20:11:09 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>