Hello Gavin,

 

I’ve found a small bug in your patch, 

 

/* If the queue entry is within avl [the number of available members] calls
from the top ... */
               if (ch && idx < avl) { <------ Bug!!!
                       if (option_debug)

                                               ast_log(LOG_DEBUG, "It's our
turn (%s).\n", qe->chan->name);

 

                        

/* If the queue entry is within avl [the number of available members] calls
from the top ... */

                        if (ch && idx <= avl) { <---- change here 

                                   if (option_debug)

                                               ast_log(LOG_DEBUG, "It's our
turn (%s).\n", qe->chan->name);

 

 
Otherwise if you have 3 queue members ready, and 3 waiting callers, only 2
instead of 3 phones are ringing at the same.
 
One other question, I’ve recognized that after applying the patch, the queue
application is distributing the calls using “last in, first out” strategy.
So if there is a large queue, the first callers in the queue are never
distributed to a queue member. Do you know anything about that behaviour,
I’m not sure if it was the same before applying the patch? 
 
 
Thanks and Regards
 
 
Markus   

 

_______________________________________________
--Bandwidth and Colocation provided by Easynews.com --

asterisk-users mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-users

Reply via email to