im.pidgin.pidgin: a1750b6bfef2b654e1620a586e4bf6c58bc3a6a9
nosnilmot at pidgin.im
nosnilmot at pidgin.im
Sat Oct 27 10:05:43 EDT 2007
-----------------------------------------------------------------
Revision: a1750b6bfef2b654e1620a586e4bf6c58bc3a6a9
Ancestor: c3509bf14bc4bd03cc7c631a2329b33df504a825
Author: nosnilmot at pidgin.im
Date: 2007-10-27T13:27:45
Branch: im.pidgin.pidgin
Modified files:
libpurple/protocols/msn/msn.c
ChangeLog:
We really don't need to display Blocked: status in the tooltip for every
buddy in a contact on MSN.
-------------- next part --------------
============================================================
--- libpurple/protocols/msn/msn.c 9de650cbbffad6d2046dda8916c784805bee13f2
+++ libpurple/protocols/msn/msn.c df3794ee7e9cefc2489b267f9f52e14b7ac5ae0b
@@ -617,7 +617,12 @@ msn_tooltip_text(PurpleBuddy *buddy, Pur
* XXX: blocked icon overlay isn't always accurate for MSN.
* XXX: This can die as soon as purple_privacy_check() knows that
* XXX: this prpl always honors both the allow and deny lists. */
- if (user)
+ /* While the above comment may be strictly correct (the privacy API needs
+ * rewriteing), purple_privacy_check() is going to be more accurate at
+ * indicating whether a particular buddy is going to be able to message
+ * you, which is the important information that this is trying to convey.
+ */
+ if (full && user)
{
purple_notify_user_info_add_pair(user_info, _("Blocked"),
((user->list_op & (1 << MSN_LIST_BL)) ? _("Yes") : _("No")));
More information about the Commits
mailing list