<p>Hi Timo,</p>
<p> this wont be available to configuration users but to other modules through API. </p>
<p>On phone now so will respond to use cases when I&#39;m back at my PC</p>
<div class="gmail_quote">On Aug 12, 2011 12:48 PM, &quot;Timo Reimann&quot; &lt;<a href="mailto:timo.reimann@1und1.de">timo.reimann@1und1.de</a>&gt; wrote:<br type="attribution">&gt; Hey,<br>&gt; <br>&gt; <br>&gt; On 12.08.2011 12:33, Jason Penton wrote:<br>
&gt;&gt; We are currently refactoring and cleaning the various IMS modules for<br>&gt;&gt; inclusion into SR, diameter_rx, diameter_cxdx, diameter_ro, etc.<br>&gt;&gt; <br>&gt;&gt; One thing we have noticed is that the use of dialog module functions<br>
&gt;&gt; would make the code alot better and cleaner, so 2 questions:<br>&gt;&gt; <br>&gt;&gt; 1. why is the Dialog module not exposing more if its methods?<br>&gt;&gt; 2. Can we put in a patch to expose the ones we require.<br>
&gt;&gt; <br>&gt;&gt; Currently, we have exposed and are using the following:<br>&gt;&gt; <br>&gt;&gt; lookup_dlg;<br>&gt;&gt; terminate_dlg;<br>&gt;&gt; get_dlg;<br>&gt;&gt; unref_dlg;<br>&gt;&gt; ref_dlg;<br>&gt; <br>&gt; I strongly opt against exporting any functions related to reference<br>
&gt; management. It&#39;s already hard to handle reference counting properly<br>&gt; inside the module; allowing configuration users to touch that part of<br>&gt; the module will likely result in all kinds of ugly bugs. IMHO, it&#39;s best<br>
&gt; to keep it internal and provide functions to whatever feature you like.<br>&gt; There&#39;s already a bunch of dialog PVs and (more recently) the very<br>&gt; generic dialog variable mechanism which allows you to do a series of things.<br>
&gt; <br>&gt; Regarding the other functions you mentioned, can you outline what your<br>&gt; use case for those is?<br>&gt; <br>&gt; <br>&gt; Cheers,<br>&gt; <br>&gt; --Timo<br>&gt; <br>&gt; _______________________________________________<br>
&gt; sr-dev mailing list<br>&gt; <a href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a><br>&gt; <a href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a><br>
</div>