[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