[Pidgin] #3319: MSN group name display support

Pidgin trac at pidgin.im
Sun Jul 5 16:33:44 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:70 VuDu]:
 > Alright, so the "non-option" turned out to be the choice made.
 > IMO, put 3 in action and start planning 4 and that requires picking up
 the patches suggested before. What do you think?

 3 yes, 4 no.

 A tag would not represent what really happened. IMO the options are:

  1. Temporarily change the alias, just like in msn-pecan
  1. Properly implement alias per-conversation
  1. Properly implement alias per-message

 Note that #1 is perfectly safe because since libpurple is single-threaded
 we don't leave a chance for the buddy list or anything else to do
 something with the temporary new alias.

 2 would probably be fine but potentially worst than 1, but what would
 really reflect what is happening is 3.

 Anything else would do more damage than good.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/3319#comment:71>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list