<p>Kevin Harwell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7873">View Change</a></p><p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p>(2 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7873/3/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/7873/3/main/loader.c@234">Patch Set #3, Line 234:</a> <code style="font-family:monospace,monospace"> * \param missing Vector to store name of any enhancer that is not running or declined.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure if the help text is wrong, the code or I am just all around misreading things :-)</p><p style="white-space: pre-wrap; word-wrap: break-word;">This says it stores names that are not running or declined, but if a module is declined the code continues in the loop and never checks dep was potentially enhanced by the declined module.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7873/3/main/loader.c@1487">Patch Set #3, Line 1487:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">              if (module_deps_reference(mod, NULL)) {<br>                       ast_log(LOG_WARNING, "Module '%s' could not be loaded due to missing dependencies.\n", resource_name);<br>                      unload_dynamic_module(mod);<br><br>                 return required ? AST_MODULE_LOAD_FAILURE : AST_MODULE_LOAD_DECLINE;<br>          }<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Not sure if this would be the best place to do it, but do you think it would be worthwhile to print out the missing dependencies here if this function was called due to a user trying to load an individual module from the cli?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7873">change 7873</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/7873"/><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: I9be08d1dd331aceadc1dcba00b804d71360b2fbb </div>
<div style="display:none"> Gerrit-Change-Number: 7873 </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: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 11 Jan 2018 23:25:30 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>