<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2: Code-Review-2</p><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Bernard.  Thanks for submitting the review.  I'll be fixing some things up for you that you might want to remember for the next time.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Reviews should always be submitted against the "base" branches (13, 16, 17, master) and never directly to the "release" branches (13.32, 16.9, etc).  They'll get moved automatically to the next release branch when the time comes.</p><p style="white-space: pre-wrap; word-wrap: break-word;">You should always submit your review against the lowest "base" branch to which it applies.  In this case, your review applies to 13 as well as 16, 17 and master.  So you can submit it against 13, then use the Gerrit "cherry-pick" function to copy it to the other branches.  If your review is large or controversial you may want to submit it only to the lowest branch and wait for at least 1 "+1" code review before cherry-picking it to the other branches.  I'm going to cherry-pick this one to 13 only for now.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I've also updated your commit message somewhat.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'm abandoning this review since it's been cherry-picked to 13.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Oops, sorry.  I'm not abandoning it, I'm moving it to 13.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14141">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14141">change 14141</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/+/14141"/><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-Change-Id: I3196803a062dd2daee4938c9a778c3810cb7e504 </div>
<div style="display:none"> Gerrit-Change-Number: 14141 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Bernard Merindol <bernard.merindol@telnowedge.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 08 Apr 2020 14:16:46 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>