comparison libpurple/protocols/msn/notification.c @ 29350:99d1b433dba0

Keep CurrentMedia in a separate object outside of MsnUser that needs to be allocated if a user has media. This makes MsnUser smaller by the size of 3 pointers (apparently) for each buddy in your buddy list. In practice it looks like less than 1% of MSN users have media set, so this should save a little bit of memory.

29349:3de19f8f5c92

	MsnSession *session;
	PurpleAccount *account;
	MsnUser *user;
	const char *passport;
	char *psm_str, *str;
	CurrentMedia media = {CURRENT_MEDIA_UNKNOWN, NULL, NULL, NULL};

	session = cmdproc->session;
	account = session->account;

	passport = cmd->params[0];

		psm_str = msn_get_psm(cmd->payload,len);
		msn_user_set_statusline(user, psm_str);
		g_free(psm_str);

		str = msn_get_currentmedia(cmd->payload, len);
		if (msn_parse_currentmedia(str, &media))
			msn_user_set_currentmedia(user, &media);
		else
			msn_user_set_currentmedia(user, NULL);
		g_free(media.title);
		g_free(media.album);
		g_free(media.artist);
		g_free(str);

	} else {
		msn_user_set_statusline(user, NULL);
		msn_user_set_currentmedia(user, NULL);
	}

	msn_user_update(user);
}

static void

29350:99d1b433dba0

	MsnSession *session;
	PurpleAccount *account;
	MsnUser *user;
	const char *passport;
	char *psm_str, *str;
	CurrentMedia *media = NULL;

	session = cmdproc->session;
	account = session->account;

	passport = cmd->params[0];

		psm_str = msn_get_psm(cmd->payload,len);
		msn_user_set_statusline(user, psm_str);
		g_free(psm_str);

		str = msn_get_currentmedia(cmd->payload, len);
		media = msn_parse_currentmedia(str);
		g_free(str);
	} else {
		msn_user_set_statusline(user, NULL);
	}
	msn_user_set_currentmedia(user, media);

	msn_user_update(user);
}

static void