<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7989">View Change</a></p><p>Patch set 2:</p><ul><li>include/asterisk/ari.h: The CHECK_ARI_MODULE_LOADED() is not needed</li><li>anymore.  Dependent modules won't start if a dependency is not</li><li>running/declined anymore right?</li></ul><ul><li>Users of ast_module_check() need to be examined for continued need.</li><li>Many are now redundant.  CHECK_ARI_MODULE_LOADED() and</li><li>CHECK_PJSIP_MODULE_LOADED() are examples.</li></ul><ul><li>Consumers of optional API's need to list the optional module in the</li><li>AST_MODULE_INFO section.  Many currently don't do this.</li></ul><p>(8 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@3826">Patch Set #2, Line 3826:</a> <code style="font-family:monospace,monospace">int AST_OPTIONAL_API_NAME(ast_agi_unregister)(struct ast_module *mod, agi_command *cmd)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">mod is now unused.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7989/2/res/res_agi.c@3892">Patch Set #2, Line 3892:</a> <code style="font-family:monospace,monospace">int AST_OPTIONAL_API_NAME(ast_agi_unregister_multiple)(struct ast_module *mod, struct agi_command *cmd, unsigned int len)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">mod is now unused</p></li><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 style="white-space: pre-wrap; word-wrap: break-word;">Revert.  While executing the command we need to ref the module or the command fails.  See agi_handle_command() which refs while executing.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7989/2/res/res_agi.c@4658">Patch Set #2, Line 4658:</a> <code style="font-family:monospace,monospace">      ast_agi_unregister(NULL, &noop_command);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Revert.  While executing the command we need to ref the module or the command fails.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7989/2/res/res_agi.c@4692">Patch Set #2, Line 4692:</a> <code style="font-family:monospace,monospace">     err |= ast_agi_register_multiple(NULL, commands, ARRAY_LEN(commands));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Revert.  While executing the command we need to ref the module or the command fails.</p></li></ul></li><li><p><a href="https://gerrit.asterisk.org/#/c/7989/2/res/res_pjsip.c">File res/res_pjsip.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.c@a2794">Patch Set #2, Line 2794:</a> <code style="font-family:monospace,monospace"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Need to keep ast_free(identifier_order).</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@1202">Patch Set #2, Line 1202:</a> <code style="font-family:monospace,monospace">/*! \brief When the res_pjsip instance is created, add an observer to it and</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">The current changes in this file should be reverted.  There are no enforced load dependencies.  This module pre-enhances res_pjsip but that cannot be expressed by the module info dependency lists.</p></li><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 style="white-space: pre-wrap; word-wrap: break-word;">(Pre-existing) This function should protect from the case where it is manually loaded AFTER res_pjsip is started.  In this case we never installed the observer and we should not unref our own module ref.</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: 2 </div>
<div style="display:none"> Gerrit-Owner: 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 04:42:07 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>