<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>2 okt 2013 kl. 16:01 skrev Daniel-Constantin Mierla <<a href="mailto:miconda@gmail.com">miconda@gmail.com</a>>:</div><br class="Apple-interchange-newline"><blockquote type="cite">
  
    <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
  
  <div bgcolor="#FFFFFF" text="#000000">
    <br>
    <div class="moz-cite-prefix">On 10/2/13 3:54 PM, Olle E. Johansson
      wrote:<br>
    </div>
    <blockquote cite="mid:225191DB-2F66-44EF-A82E-A3E716CAC890@edvina.net" type="cite">
      <pre wrap="">A few comments:

I think the module name should be "queue". The moh-part is a just a nice way of handling the queue...</pre>
    </blockquote>
    I don't like "queue" alone, because it is not suggestive for the
    functionality. It can be callqueue or something else that contains
    queue if the current name is no longer preferred, but not just
    'queue'.<br></div></blockquote>I see. +1 for "callqueue".</div><div><br></div><div>/O<br><blockquote type="cite"><div bgcolor="#FFFFFF" text="#000000">
    <br>
    Cheers,<br>
    Daniel<br>
    <blockquote cite="mid:225191DB-2F66-44EF-A82E-A3E716CAC890@edvina.net" type="cite">
      <pre wrap="">
>From reading the docs:

- The example on 3.4. moh_maxcalls illustrates usage of mohdir.

- Change "from the main (request) route" to "a request route". Routes can call routes.

I would love for this module to expose the queue in an xavp (as an alternative to the database) so I can manipulate weights and stuff. Queues need management. I would also like to be able to pick a call out of the queue at any point, regardless of position. But that's a future change. This is a very good contribution.

I don't think naming audio files after payload number is a good scheme. Payload numbers are dynamic for all new codecs and one should use the name tags. I think we should simply copy the file name extensions used by Asterisk for the codecs, like "alaw" for alaw. Or the names used in rtpmap. One of them - but not the numbers, that is not extensible enough as they are no longer assigned. Opus will not get a fixed number, just a name tag.

/O




</pre>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
sr-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a>
<a class="moz-txt-link-freetext" href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a>
</pre>
    </blockquote>
    <br>
    <pre class="moz-signature" cols="72">-- 
Daniel-Constantin Mierla - <a class="moz-txt-link-freetext" href="http://www.asipto.com/">http://www.asipto.com</a>
<a class="moz-txt-link-freetext" href="http://twitter.com/#!/miconda">http://twitter.com/#!/miconda</a> - <a class="moz-txt-link-freetext" href="http://www.linkedin.com/in/miconda">http://www.linkedin.com/in/miconda</a>
Kamailio Advanced Trainings - Berlin, Nov 25-28; Miami, Nov 18-20, 2013
  - more details about Kamailio trainings at <a class="moz-txt-link-freetext" href="http://www.asipto.com/">http://www.asipto.com</a> -
</pre>
  </div>

_______________________________________________<br>sr-dev mailing list<br><a href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a><br>http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev<br></blockquote></div><br></body></html>