<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 3:</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 3:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Sean Bright, Jaco Kroon: Would either of you like to take this on to update it?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Sure, can take a shot, but this is not even on my priority radar at all currently.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Did Leandro abandon this?  I honestly feel I'd rather have him finish this if possible, even if we do need to guide a bit.  It's been quiet for a month though ... so guessing one of us will have to take this over.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Ah he responded over email:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I am sorry, but currently I am on tight schedules, so I have no free time to dedicate at this issue. If you have someone able to take care of this issue, I'll be glad.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;"><br>How thoroughly do you want this?  Ie, is it worthwhile to create a char parenthesis[] based stack to match different types of brackets and handle that as an error condition too, or is mere counting adequate?  Basically what I'm thinking is create a char parens[20]; paren_depth = 0;</p><p style="white-space: pre-wrap; word-wrap: break-word;">When we encounter on of [({ we push it onto the stack (or error out if depth already == 20), and if we encounter ])} we pop and make sure match.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If we encounter = we ignore unless depth == 0.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I haven't looked into this so can't comment. I'm just doing Gerrit clean up and seeing where things stand.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/15331">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15331">change 15331</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/+/15331"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: I7cd3bc6118033530ad00ae4b6bdad97b6b349b6f </div>
<div style="display:none"> Gerrit-Change-Number: 15331 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Leandro Dardini <ldardini@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-CC: Jaco Kroon <jaco@uls.co.za> </div>
<div style="display:none"> Gerrit-CC: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Mar 2021 09:25:01 +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>