[asterisk-commits] mmichelson: branch mmichelson/queue_refcount r81450 - /team/mmichelson/queue_...

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Tue Sep  4 13:50:12 2007
New Revision: 81450

URL: http://svn.digium.com/view/asterisk?view=rev&rev=81450
Log:
A couple of changes to fix some errors, but moreso getting rid of a ridiculous comment I had.
Thanks go to Russell for pointing it out to me...and everyone else...


Modified:
    team/mmichelson/queue_refcount/apps/app_queue.c

Modified: team/mmichelson/queue_refcount/apps/app_queue.c
URL: http://svn.digium.com/view/asterisk/team/mmichelson/queue_refcount/apps/app_queue.c?view=diff&rev=81450&r1=81449&r2=81450
==============================================================================
--- team/mmichelson/queue_refcount/apps/app_queue.c (original)
+++ team/mmichelson/queue_refcount/apps/app_queue.c Tue Sep  4 13:50:12 2007
@@ -399,7 +399,7 @@
 	int memberdelay;                    /*!< Seconds to delay connecting member to caller */
 	int autofill;                       /*!< Ignore the head call status and ring an available agent */
 	
-	ao2_container members;              /*!< DOODOO!! :*)*::: :(){:|:&};: */
+	ao2_container members;              /*!< The list of members in the queue */ 
 	int membercount;					/*!< Number of members in queue */
 	struct queue_ent *head;             /*!< Head of the list of callers */
 };
@@ -513,7 +513,7 @@
 
 	ast_mutex_lock(&q->lock);
 	i = ao2_iterator_init(q->members, 0);
-	while (member = ao2_iterator_next(&i){
+	while ((member = ao2_iterator_next(&i))){
 		if (max_penalty && (member->penalty > max_penalty)) {
 			ao2_ref(member, -1);
 			continue;
@@ -1008,8 +1008,7 @@
 	ao2_iterator mem_iter;
 
 	mem_iter = ao2_iterator_init(q->members, 0);
-	while (ao2_iterator_next(&mem_iter) {
-		next = curm->next;
+	while ((curm = ao2_iterator_next(&mem_iter))) {
 		if (all || !curm->dynamic) {
 			ao2_unlink(curm);
 			remove_from_interfaces(curm->interface);