<p>Jaco Kroon <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/15448">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">func_odbc:  Introduce minargs config and expose ARGC in addition to ARGn.<br><br>minargs enables enforcing of minimum count of arguments to pass to<br>func_odbc, so if you're unconditionally using ARG1 through ARG4 then<br>this should be set to 4.  func_odbc will generate an error in this case,<br>so for example<br><br>[FOO]<br>minargs = 4<br><br>and ODBC_FOO(a,b,c) in dialplan will now error out instead of using a<br>potentially leaked ARG4 from Gosub().<br><br>ARGC is needed if you're using optional argument, to verify whether or<br>not an argument has been passed, else it's possible to use a leaked ARGn<br>from Gosub (app_stack).  So now you can safely do<br>${IF($[${ARGC}>3]?${ARGV}:default value)} kind of thing.<br><br>Change-Id: I6ca0b137d90b03f6aa9c496991f6cbf1518f6c24<br>Signed-off-by: Jaco Kroon <jaco@uls.co.za><br>---<br>M configs/samples/func_odbc.conf.sample<br>M funcs/func_odbc.c<br>2 files changed, 40 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/48/15448/4</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15448">change 15448</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/+/15448"/><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: I6ca0b137d90b03f6aa9c496991f6cbf1518f6c24 </div>
<div style="display:none"> Gerrit-Change-Number: 15448 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Jaco Kroon <jaco@uls.co.za> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>