/pidgin/main: 77d8af8354e7: I think it's more appropriate to pus...

Mark Doliner mark at kingant.net
Tue Sep 11 20:32:07 EDT 2012


Changeset: 77d8af8354e7672756bb54db1cab8723be542e48
Author:	 Mark Doliner <mark at kingant.net>
Date:	 2012-09-11 17:32 -0700
Branch:	 default
URL: http://hg.pidgin.im/pidgin/main/rev/77d8af8354e7

Description:

I think it's more appropriate to push to the head of the queue rather than
the tail, here.  Because we're not adding a new request, we're re-adding
a request that we just popped but then realized that we can't handle right
now.

I believe this matches the behavior from before the switch to GQueue in
2375ffdbb565.

diffstat:

 libpurple/dnsquery.c |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff --git a/libpurple/dnsquery.c b/libpurple/dnsquery.c
--- a/libpurple/dnsquery.c
+++ b/libpurple/dnsquery.c
@@ -608,7 +608,7 @@ handle_next_queued_request(void)
 		if (number_of_dns_children >= MAX_DNS_CHILDREN)
 		{
 			/* Apparently all our children are busy */
-			g_queue_push_tail(queued_requests, query_data);
+			g_queue_push_head(queued_requests, query_data);
 			return;
 		}
 



More information about the Commits mailing list