<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8390">View Change</a></p><p>Patch set 1:</p><p>(4 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/8390/1/include/asterisk/module.h">File include/asterisk/module.h:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8390/1/include/asterisk/module.h@295">Patch Set #1, Line 295:</a> <code style="font-family:monospace,monospace">   AST_MODPRI_CORE =               40,  /*!< A core module originally meant to start between preload and load. */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I don't think so, I'm not aware of any dependencies between timing modules </blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think for safety we should make room by adding the 10 as you have indicated.</p></li></ul></li><li><p><a href="https://gerrit.asterisk.org/#/c/8390/1/main/cel.c">File main/cel.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8390/1/main/cel.c@1568">Patch Set #1, Line 1568:</a> <code style="font-family:monospace,monospace">           return AST_MODULE_LOAD_FAILURE;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I think this is inconsistent.  We do not run unload_module on 'core stop no</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Many modules actually do cleanup as best they can when they are aborting the load.  res_pjsip.c and chan_sip.c do.</p></li></ul></li><li><p><a href="https://gerrit.asterisk.org/#/c/8390/1/main/loader.c">File main/loader.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8390/1/main/loader.c@509">Patch Set #1, Line 509:</a> <code style="font-family:monospace,monospace">              mod = ast_calloc(1, sizeof(*mod) + strlen(info->name) + 1);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Hmm I think I'll have to ast_std_calloc here, then when built-in modules ar</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">You don't have to recreate it.  You can use the builtin flag to release it using ast_std_free if set and ast_free otherwise.</p></li></ul></li><li><p><a href="https://gerrit.asterisk.org/#/c/8390/1/res/res_config_sqlite.c">File res/res_config_sqlite.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8390/1/res/res_config_sqlite.c@1780">Patch Set #1, Line 1780:</a> <code style="font-family:monospace,monospace"> .requires = "cdr",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This module is funny as it registers a realtime driver, loads config and re</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Then the dependencies should be:</p><p style="white-space: pre-wrap; word-wrap: break-word;">.requires = "extconfig",<br>.optional_modules = "cdr",</p><p style="white-space: pre-wrap; word-wrap: break-word;">The module was just deprecated in the middle of February so the support level being wrong is understandable.  This points out that the recently deprecated modules may not have updated the AST_MODULE_INFO status to match.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8390">change 8390</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/8390"/><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: I371a9a45064f20026c492623ea8062d02a1ab97f </div>
<div style="display:none"> Gerrit-Change-Number: 8390 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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, 08 Mar 2018 19:43:38 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>