[Pidgin] #4598: Plus! color tags

[Pidgin]

#4598: Plus! color tags
-----------------------------------------+----------------------------------
 Reporter:  daniele_athome               |        Owner:              
     Type:  patch                        |       Status:  new         
Milestone:  Patches Needing Improvement  |    Component:  pidgin (gtk)
  Version:  2.3.1                        |   Resolution:              
 Keywords:  pidgin blist markup          |  
-----------------------------------------+----------------------------------

Comment(by elb):

 This patch looks like a memory leak waiting to happen -- it strdup()s a
 string returned from a signal and discards the signal result.  I would
 generally expect the signal handler in a situation like this to have
 allocated the string it wished to return.

 It's also not clear to me why this signal is named "drawing-buddy" and
 takes a buddy as an argument, but seems to be utilized only to return
 markup for the buddy's username.  A more specific signal name which
 indicates this rather limited scope would be appropriate here.

 It looks to me like the whole purpose of this plugin is to handle MSN's
 ill-conceived username markup language; this is something which should
 properly be handled by the MSN protocol plugin.  Perhaps a signal should
 be emitted from the MSN protocol plugin to the effect of msn-buddy-parse-
 markup, with a default implementation of simply stripping these codes from
 the nickname.

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