<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://reviewboard.asterisk.org/r/1887/">https://reviewboard.asterisk.org/r/1887/</a>
     </td>
    </tr>
   </table>
   <br />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">All in all, this seems like it&#39;s good to go aside from the minor squabbles above.  I&#39;m a little wary to give ship it since I&#39;ve allowed queue stuff for that kinda jacked up some features in queues. I looked through the code for a good while and checked most for the kind of stuff that tends to trip me up when working though list traversal and such... didn&#39;t find anything of interest there. If no one else has given this any commentary by tomorrow, I&#39;ll give it another pass and if I haven&#39;t found anything too grievous, I&#39;ll go ahead and give it a pass.</pre>
 <br />







<p>- jrose</p>


<br />
<p>On April 26th, 2012, 9:42 a.m., Olle E Johansson wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Asterisk Developers.</div>
<div>By Olle E Johansson.</div>


<p style="color: grey;"><i>Updated April 26, 2012, 9:42 a.m.</i></p>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Today, when a prompt is being played to a call waiting in the queue and an agent becomes available, the agent will not get the call until the prompt is finished. Both customer and agent is kept waiting.

With this rather big piece of code, we attach a generator to play prompts. The generator can, like all generators, be stopped at any time so that the agent (queue member) can get the call immediately.

The generator is now placed in app_queue, but could propably be moved somewhere else. It also changes functionality in main/say.c in order to be able to place those prompts in the same prompt queue for background processing. The same generator could be used to sevice conference bridges and maybe be added as a dialplan function at some point for background playlists.
</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Quite a lot of testing in our environment during the rather long time we&#39;ve been testing this. Customer is happy.




There was some complaints from a bowl of petunias, which made me a bit surprised.</pre>
  </td>
 </tr>
</table>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="https://issues.asterisk.org/jira/browse/19795">19795</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>/trunk/apps/app_queue.c <span style="color: grey">(364000)</span></li>

 <li>/trunk/configs/queues.conf.sample <span style="color: grey">(364000)</span></li>

 <li>/trunk/include/asterisk/channel.h <span style="color: grey">(364000)</span></li>

 <li>/trunk/include/asterisk/file.h <span style="color: grey">(364000)</span></li>

 <li>/trunk/main/asterisk.dynamics <span style="color: grey">(364000)</span></li>

 <li>/trunk/main/file.c <span style="color: grey">(364000)</span></li>

 <li>/trunk/main/say.c <span style="color: grey">(364000)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/1887/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>