<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7989">View Change</a></p><p>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;">(8 comments)</p><ul><li>include/asterisk/ari.h: The CHECK_ARI_MODULE_LOADED() is not</li><li>needed</li><li>anymore.  Dependent modules won't start if a dependency is not</li><li>running/declined anymore right?</li></ul></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;"><br>ARI modules do not use optional_modules to express the dependency so that is correct.  So the macro and all calls are no longer needed.</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br> > * Users of ast_module_check() need to be examined for continued<br> > need.<br> > Many are now redundant.  CHECK_ARI_MODULE_LOADED() and<br> > CHECK_PJSIP_MODULE_LOADED() are examples.</pre><p style="white-space: pre-wrap; word-wrap: break-word;"><br>I'm expanding this patch to include redundant calls to ast_module_check so now that function is only used at the request of func_module, AMI or ARI users.</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br> > * Consumers of optional API's need to list the optional module in<br> > the<br> > AST_MODULE_INFO section.  Many currently don't do this.</pre><p style="white-space: pre-wrap; word-wrap: break-word;"><br>I'm auditing all includes of asterisk/optional_api.h and post what I find as a separate patch.</p><p>(2 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7989/2/res/res_agi.c">File res/res_agi.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7989/2/res/res_agi.c@4641">Patch Set #2, Line 4641:</a> <code style="font-family:monospace,monospace">       if (ast_agi_register(NULL, &noop_command) == 0) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Revert.  While executing the command we need to ref the module or the comma</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Sigh.. always embarrassed when one of my patches break another and I miss it.</p></li></ul></li><li><p><a href="https://gerrit.asterisk.org/#/c/7989/2/res/res_pjsip_config_wizard.c">File res/res_pjsip_config_wizard.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7989/2/res/res_pjsip_config_wizard.c@1215">Patch Set #2, Line 1215:</a> <code style="font-family:monospace,monospace">static void instance_destroying_observer(const char *name, struct ast_sorcery *sorcery)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">(Pre-existing) This function should protect from the case where it is manua</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure what to do about this, but since it's a pre-existing issue it should be addressed in all branches?  Either way I'm reverting changes to this file and calling this out of scope.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7989">change 7989</a>. To unsubscribe, 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/7989"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I332a6e8383d4c72c8e89d988a184ab8320c4872e </div>
<div style="display:none"> Gerrit-Change-Number: 7989 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 18 Jan 2018 14:27:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>