The issue opened for this &quot;janitor&quot; is: 0011096        <br><br>Thanks<br><br><div><span class="gmail_quote">On 10/25/07, <b class="gmail_sendername">Eliel Sardanons</b> &lt;<a href="mailto:eliel@eliel.com.ar">eliel@eliel.com.ar
</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Ok Moises,<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; I will submit the first patch and open an issue to start working on this janitor.
<div><span class="e" id="q_115d8c17b9f827d2_1"><br><br><br><br><div><span class="gmail_quote">On 10/25/07, <b class="gmail_sendername">Moises Silva</b> &lt;<a href="mailto:moises.silva@gmail.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

moises.silva@gmail.com</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Eliel,<br><br>Let us know if you open a bug in mantis for this in order to put
<br>changes there as well.<br><br>Thanks<br><br>Moy<br><br>On 10/25/07, Eliel Sardanons &lt;<a href="mailto:eliel@eliel.com.ar" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">eliel@eliel.com.ar</a>
&gt; wrote:<br>&gt; This are load_module()&#39;s return values, I am not changing nothing,&nbsp;&nbsp;this is
<br>&gt; just the way &quot;it should be&quot;, read chan_sip.c&nbsp;&nbsp;and you will notice how<br>&gt; &#39;return AST_MODULE_LOAD_SUCCESS&#39; is being used.<br>&gt; But if you take a look at some modules (chan_gtalk.c), they don&#39;t use this
<br>&gt; return values defined in the enum, they use just the values &quot;0, -1,1,2&quot; and<br>&gt; we should change everything to start using the names<br>&gt; AST_MODULE_LOAD_SUCCESS, *_DECLIE, *_SKIP, etc.<br>&gt;<br>

&gt; Sample:<br>&gt; - return 0;<br>&gt; + return AST_MODULE_LOAD_SUCCESS;<br>&gt;<br>&gt;<br>&gt;<br>&gt; On 10/25/07, Tzafrir Cohen &lt;<a href="mailto:tzafrir.cohen@xorcom.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
tzafrir.cohen@xorcom.com</a> &gt; wrote:<br>
&gt; &gt; On Thu, Oct 25, 2007 at 02:09:02PM -0300, Eliel Sardanons wrote:<br>&gt; &gt; &gt; I think we could start a new janitor project to replace every &#39;return&#39;<br>&gt; on<br>&gt; &gt; &gt; load_module&#39;s with:
<br>&gt; &gt; &gt;<br>&gt; &gt; &gt; enum ast_module_load_result {<br>&gt; &gt; &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AST_MODULE_LOAD_SUCCESS = 0,&nbsp;&nbsp;&nbsp;&nbsp;/*!&lt; Module loaded and<br>&gt; configured */<br>&gt; &gt; &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AST_MODULE_LOAD_DECLINE = 1,&nbsp;&nbsp;&nbsp;&nbsp;/*!&lt; Module is not configured */
<br>&gt; &gt; &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AST_MODULE_LOAD_SKIP = 2,&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; /*!&lt; Module was skipped for some<br>&gt; &gt; &gt; reason */<br>&gt; &gt; &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AST_MODULE_LOAD_FAILURE = -1,&nbsp;&nbsp; /*!&lt; Module could not be loaded<br>&gt; &gt; &gt; properly */
<br>&gt; &gt; &gt; };<br>&gt; &gt;<br>&gt; &gt; What do those return values mean? Does a return value of -1 mean that<br>&gt; &gt; Asterisk should die?<br>&gt; &gt;<br>&gt; &gt; --<br>&gt; &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Tzafrir Cohen
<br>&gt; &gt; icq#16849755&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;jabber:<a href="mailto:tzafrir.cohen@xorcom.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">tzafrir.cohen@xorcom.com</a><br>&gt; &gt; +972-50-7952406&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; mailto:
<a href="mailto:tzafrir.cohen@xorcom.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">tzafrir.cohen@xorcom.com
</a><br>&gt; &gt; <a href="http://www.xorcom.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://www.xorcom.com</a><br>&gt; <a href="http://iax:guest@local.xorcom.com/tzafrir" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
iax:guest@local.xorcom.com/tzafrir</a><br>&gt; &gt;<br>&gt; &gt; _______________________________________________
<br>&gt; &gt; --Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://www.api-digital.com--</a><br>&gt; &gt;<br>&gt; &gt; asterisk-dev mailing list
<br>&gt; &gt; To UNSUBSCRIBE or update options visit:
<br>&gt; &gt;&nbsp;&nbsp;&nbsp;&nbsp;<a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br>&gt; &gt;
<br>&gt;<br>&gt;<br>&gt;<br>&gt; --<br>&gt; Eliel Sardaņons<br>&gt; _______________________________________________
<br>&gt; --Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://www.api-digital.com--</a><br>&gt;<br>&gt; asterisk-dev mailing list
<br>&gt; To UNSUBSCRIBE or update options visit:<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp;
<a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br>&gt;<br><br><br>--<br>&quot;Within C++, there is a much smaller and cleaner language struggling
<br>
to get out.&quot;<br><br>_______________________________________________<br>--Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
http://www.api-digital.com--</a><br><br>asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>&nbsp;&nbsp; <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://lists.digium.com/mailman/listinfo/asterisk-dev
</a><br></blockquote></div><br><br clear="all"><br></span></div><span class="sg">-- <br>
Eliel Sardaņons
</span></blockquote></div><br><br clear="all"><br>-- <br>Eliel Sardaņons