<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;">we should also check for an error result from all calls to res_crypto functions, return an error from the dialplan functions if appropriate</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">What does this mean for me? (a) Do I have to add that, to get this change passed, (b) do you take that over in a new change (as feature request), (c) or do you want to take over this change here and add that? Guys, think about the contributor and how to ease his live. Not everyone is able to git, git review, or even git rebase. Tell him what *exactly* is expected from the contributor, how he and his change can continue, *and* why you cannot do it yourself. That is not that difficult if you try to put yourself into the view of the contributor.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Furthermore, it would be cool if the Asterisk Team could feedback in the master branch and not in one of the cherry-picks . Although I get that E-mail notifications, I had to search now in which branch the feedback actually was. Yes, of course, if the cherry-picks are different, feedback there. However, for a change which is the same for all cherry picks, please, feedback in the master branch. That saves time. And time is code.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Corey is a community member and contributor, just like you.</p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4; color: #000000;">Code-Review +1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14010">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14010">change 14010</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/+/14010"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 17 </div>
<div style="display:none"> Gerrit-Change-Id: I0b99b8468cbeb3b0eab23069cbd64062ef885ffc </div>
<div style="display:none"> Gerrit-Change-Number: 14010 </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: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-CC: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 31 Mar 2020 13:02:51 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>