comparison libpurple/protocols/msn/session.c @ 24328:5b5c21b60690

Get rid of an unnecessary if statement, and don't try to unref a null icon when you don't have an icon set on msn.

24327:38f4670c3daa

	{
		purple_debug_error("msn", "This shouldn't happen\n");
		g_return_val_if_reached(FALSE);
	}

	if (msn_notification_connect(session->notification, host, port))
	{
		return TRUE;
	}

	return FALSE;
}

void
msn_session_disconnect(MsnSession *session)
{

	account = session->account;
	gc = purple_account_get_connection(account);

	img = purple_buddy_icons_find_account_icon(session->account);
	msn_user_set_buddy_icon(session->user, img);
	purple_imgstore_unref(img);

	session->logged_in = TRUE;

	msn_change_status(session);

24328:5b5c21b60690

	{
		purple_debug_error("msn", "This shouldn't happen\n");
		g_return_val_if_reached(FALSE);
	}

	return msn_notification_connect(session->notification, host, port);
}

void
msn_session_disconnect(MsnSession *session)
{

	account = session->account;
	gc = purple_account_get_connection(account);

	img = purple_buddy_icons_find_account_icon(session->account);
	/* TODO: Do we really want to call this if img is NULL? */
	msn_user_set_buddy_icon(session->user, img);
	if (img != NULL)
		purple_imgstore_unref(img);

	session->logged_in = TRUE;

	msn_change_status(session);