<p>Alexander Traud <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6807">View Change</a></p><p>Patch set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I have the time. I simply do not do it. My bug fix changes the behavior of those who *have not* cared about keep_alive_interval. Otherwise they would have seen this bug, this second keep-alive coming from somewhere, before and reported it. Joshua’s proposal changes the behavior for those who *have* changed keep_alive_interval. In both alternatives, there are corner cases, which *could* be serious ripple-effects for others. In both alternatives. I am not convinced that one is better than the other. I am even convinced that there are no ripple-effects with my proposal.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Furthermore, default values should be good, reasonable defaults and I do not think a keep_alive_interval from a UAS is a good behavior. It is such an unexpected behavior, that one of my devices crashes. Therefore Joshua’s proposal is not a good default. Finally, I do not benefit from this change, I even face drawbacks if I go for Joshua’s proposal. My patch works for me. Therefore, there is no motivation to go for this.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6807">change 6807</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/6807"/><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: I27c7a87c882b02f56ff3b739630d9b9e5912cc68 </div>
<div style="display:none"> Gerrit-Change-Number: 6807 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 19 Oct 2017 11:07:24 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>