[Pidgin] #3319: MSN group name display support
Pidgin
trac at pidgin.im
Sun Jul 5 15:24:49 EDT 2009
#3319: MSN group name display support
----------------------------+-----------------------------------------------
Reporter: manphiz | Owner: khc
Type: rejected_patch | Status: new
Milestone: | Component: MSN
Version: 2.2.0 | Resolution:
Keywords: msn group |
----------------------------+-----------------------------------------------
Comment(by felipec):
Replying to [comment:68 VuDu]:
> I don't understand why somewhere in the middle of this ticket the word
"tag" started being used.
> We always had (and should always have) "(timestamp) alias: message". The
only thing that changed is that now msn_message_get_attr(msg,
"P4-Context") might not be NULL, right? Why entangle things?
> IMO, if it's there use it... if not, move on.
Correct, and it's a really really minor change:
http://github.com/felipec/msn-
pecan/commit/1386c1f81fb3214d67f1de9b3349b85bbd79adc9
P4-Context is meant to be a per-message nick; whatever Pidgin devs have
decided to do with it is wrong.
--
Ticket URL: <http://developer.pidgin.im/ticket/3319#comment:69>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list