I think we could start a new janitor project to replace every &#39;return&#39; on load_module&#39;s with:<br><br>enum ast_module_load_result {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AST_MODULE_LOAD_SUCCESS = 0,&nbsp;&nbsp;&nbsp; /*!&lt; Module loaded and configured */
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AST_MODULE_LOAD_DECLINE = 1,&nbsp;&nbsp;&nbsp; /*!&lt; Module is not configured */<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AST_MODULE_LOAD_SKIP = 2,&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; /*!&lt; Module was skipped for some reason */<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AST_MODULE_LOAD_FAILURE = -1,&nbsp;&nbsp; /*!&lt; Module could not be loaded properly */
<br>};<br><br>If you agree I will start working on this. I also think there are some modules like chan_gtalk and chan_jingle that doesn&#39;t make use of the proper return values.<br clear="all"><br>-- <br>Eliel Sardaņons