I don't think the comment made sense, this is probably what it meant

Mon, 26 Oct 2009 06:53:35 +0000

author
Ka-Hing Cheung <khc@pidgin.im>
date
Mon, 26 Oct 2009 06:53:35 +0000
changeset 28749
601a10ad0be1
parent 28748
53f15767337e
child 28750
185fe805c9d0

I don't think the comment made sense, this is probably what it meant
to be. It's only checking status type but not the status message, so
it can match the "wrong" status if you have substatuses of different
messages.

I am not sure if that's a feature or bug though. Seems like a bug to
me because it prevents you from setting certain messages if you
already have similar transient status.

Oh, and why does this matching only work on transient statuses anyway.

pidgin/gtkstatusbox.c file | annotate | diff | comparison | revisions
--- a/pidgin/gtkstatusbox.c	Mon Oct 26 01:11:53 2009 +0000
+++ b/pidgin/gtkstatusbox.c	Mon Oct 26 06:53:35 2009 +0000
@@ -2435,7 +2435,7 @@
 						purple_strequal(ss_msg, message))
 					{
 						gboolean found = FALSE;
-						/* The currently enabled accounts must have substatuses for all the active accts */
+						/* this status must have substatuses for all the active accts */
 						for(tmp = active_accts; tmp != NULL; tmp = tmp->next) {
 							PurpleAccount *acct = tmp->data;
 							PurpleSavedStatusSub *sub = purple_savedstatus_get_substatus(ss, acct);

mercurial