<p> Attention is currently required from: Alexander Traud. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17334">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17334?tab=comments">Patch Set #2:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The problem is not that easy. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I understand the review process as well as the concept of separation of concerns. In general I agree a patch should be a targeted fix, but depending on the circumstances I don't see the harm in minor breakages to that opinion.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In this case the reason I asked is because it seemed like it would be a small, partially related change to improve the documentation in a section of code you were already working in.</p><p style="white-space: pre-wrap; word-wrap: break-word;">However, if for any reason you feel uncomfortable, unwilling, or unable to make to make a particular modification, or submit a patch, please and always feel free to explain why and not do so. Reasonable opinions are always considered. For example, your reasons given here would have been, and are perfectly valid in my opinion for not wanting to make the requested modification.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17334">change 17334</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/c/asterisk/+/17334"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4687857b9d56e6f44fd440b73af156691660202e </div>
<div style="display:none"> Gerrit-Change-Number: 17334 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 29 Nov 2021 19:55:15 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Comment-In-Reply-To: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>