<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7873">View Change</a></p><p>Patch set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">This commit is mostly functional.  Some modules could be missing dependencies (if I can't built them).  That could mean those modules might be loaded in the incorrect order, though this would also indicate the load_pri of those modules is wrong.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I have not run the testsuite on this.  Most of my testing has involved debugging the order of calls to ast_module_info->load.  Some modules of equal priority may load in a different order, but the order between such modules is undefined anyways.  I've also verified that basic startup + shutdown of Asterisk does not produce REF_DEBUG leaks, so all modules are able to unload (no circular references).</p><p style="white-space: pre-wrap; word-wrap: break-word;">At last check this did not cause any modules to load before modules of lower priority.  Initially a couple modules were loaded out of "load_pri" order, in both cases the load_pri was incorrect and has already been fixed in all branches.</p><ul style="list-style: none; padding-left: 20px;"></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: 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-Comment-Date: Mon, 08 Jan 2018 22:34:21 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>