pidgin: 68633ca2: I don't think the comment made sense, th...

[khc at pidgin.im]

-----------------------------------------------------------------
Revision: 68633ca2b068e04c858f91556d19ee2fa51c2edf
Ancestor: 53dd1003ec6cbd5ede404f53721feaf5e7246870
Author: khc at pidgin.im
Date: 2009-10-26T06:53:35
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/68633ca2b068e04c858f91556d19ee2fa51c2edf

Modified files:
        pidgin/gtkstatusbox.c

ChangeLog: 

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.


-------------- next part --------------
============================================================
--- pidgin/gtkstatusbox.c	e7dbc2857f3e967646f64d3bb48af6bb4d388ad3
+++ pidgin/gtkstatusbox.c	64782dab34af24c7e15a2f033543ca6193f883d8
@@ -2435,7 +2435,7 @@ activate_currently_selected_status(Pidgi
 						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);