comparison pidgin/gtkstatusbox.c @ 28818:2c1677e123dc

there's no reason for message_changed to use g_utf8_collate, and also no reason why it can't be replaced by purple_strequal

28817:e35043e96a1a

	if (!typing_stock_ids[++status_box->typing_index])
		status_box->typing_index = 0;
	pidgin_status_box_refresh(status_box);
}

static gboolean
message_changed(const char *one, const char *two)
{
	if (one == NULL && two == NULL)
		return FALSE;

	if (one == NULL || two == NULL)
		return TRUE;

	return (g_utf8_collate(one, two) != 0);
}

static void
activate_currently_selected_status(PidginStatusBox *status_box)
{
	PidginStatusBoxItemType type;
	gpointer data;

			id = purple_status_type_get_id(acct_status_type);

			if (strncmp(id, purple_status_get_id(status), strlen(id)) == 0)
			{
				/* Selected status and previous status is the same */
				if (!message_changed(message, purple_status_get_attr_string(status, "message")))
				{
					PurpleSavedStatus *ss = purple_savedstatus_get_current();
					/* Make sure that statusbox displays the correct thing.
					 * It can get messed up if the previous selection was a
					 * saved status that wasn't supported by this account */

		} else {
			saved_status = purple_savedstatus_get_current();
			if (purple_savedstatus_get_type(saved_status) == primitive &&
			    !purple_savedstatus_has_substatuses(saved_status))
			{
				if (!message_changed(purple_savedstatus_get_message(saved_status), message))
					changed = FALSE;
			}
		}

		if (changed)

				for (; iter != NULL; iter = iter->next) {
					PurpleSavedStatus *ss = iter->data;
					const char *ss_msg = purple_savedstatus_get_message(ss);
					if ((purple_savedstatus_get_type(ss) == primitive) && purple_savedstatus_is_transient(ss) &&
						purple_savedstatus_has_substatuses(ss) && /* Must have substatuses */
						!message_changed(ss_msg, message))
					{
						gboolean found = FALSE;
						/* The currently enabled accounts must have substatuses for all the active accts */
						for(tmp = active_accts; tmp != NULL; tmp = tmp->next) {
							PurpleAccount *acct = tmp->data;

		id = purple_status_type_get_id(status_type);

		if (strncmp(id, purple_status_get_id(status), strlen(id)) == 0)
		{
			/* Selected status and previous status is the same */
			if (!message_changed(message, purple_status_get_attr_string(status, "message")))
				changed = FALSE;
		}

		if (changed)
		{

28818:2c1677e123dc

	if (!typing_stock_ids[++status_box->typing_index])
		status_box->typing_index = 0;
	pidgin_status_box_refresh(status_box);
}

static void
activate_currently_selected_status(PidginStatusBox *status_box)
{
	PidginStatusBoxItemType type;
	gpointer data;

			id = purple_status_type_get_id(acct_status_type);

			if (strncmp(id, purple_status_get_id(status), strlen(id)) == 0)
			{
				/* Selected status and previous status is the same */
				if (purple_strequal(message, purple_status_get_attr_string(status, "message")))
				{
					PurpleSavedStatus *ss = purple_savedstatus_get_current();
					/* Make sure that statusbox displays the correct thing.
					 * It can get messed up if the previous selection was a
					 * saved status that wasn't supported by this account */

		} else {
			saved_status = purple_savedstatus_get_current();
			if (purple_savedstatus_get_type(saved_status) == primitive &&
			    !purple_savedstatus_has_substatuses(saved_status))
			{
				if (purple_strequal(purple_savedstatus_get_message(saved_status), message))
					changed = FALSE;
			}
		}

		if (changed)

				for (; iter != NULL; iter = iter->next) {
					PurpleSavedStatus *ss = iter->data;
					const char *ss_msg = purple_savedstatus_get_message(ss);
					if ((purple_savedstatus_get_type(ss) == primitive) && purple_savedstatus_is_transient(ss) &&
						purple_savedstatus_has_substatuses(ss) && /* Must have substatuses */
						purple_strequal(ss_msg, message))
					{
						gboolean found = FALSE;
						/* The currently enabled accounts must have substatuses for all the active accts */
						for(tmp = active_accts; tmp != NULL; tmp = tmp->next) {
							PurpleAccount *acct = tmp->data;

		id = purple_status_type_get_id(status_type);

		if (strncmp(id, purple_status_get_id(status), strlen(id)) == 0)
		{
			/* Selected status and previous status is the same */
			if (purple_strequal(message, purple_status_get_attr_string(status, "message")))
				changed = FALSE;
		}

		if (changed)
		{