[Pidgin] #4052: extra 'buddy-status-changed' signals

Pidgin trac at pidgin.im
Fri Dec 21 00:34:58 EST 2007


#4052: extra 'buddy-status-changed' signals
-------------------------+--------------------------------------------------
  Reporter:  andy753421  |       Owner:  MarkDoliner 
      Type:  defect      |      Status:  new         
  Priority:  minor       |   Milestone:              
 Component:  AIM         |     Version:  2.2.2       
Resolution:              |    Keywords:  oscar status
   Pending:  0           |  
-------------------------+--------------------------------------------------
Comment (by tmm1):

 To summarize, '''purple_blist_update_buddy_status''' is called by both
 '''purple_got_infoblock''' and '''purple_prpl_got_user_status'''.
 '''purple_prpl_got_user_status''' uses the value of
 '''aim_userinfo_t->status''', which contains no HTML.
 '''purple_got_infoblock''' uses '''aim_userinfo_t->away''', which is the
 HTML version of the away message.

 The problems is that the first few times '''purple_got_infoblock''' emits
 the signal, it does so with the old away message. This happens because
 '''purple_got_infoblock'''s caller, '''aim_locate_adduserinfo''' only
 updates part of the userinfo struct that have been changed. Since the new
 incoming userinfo has away set to NULL, the old message isn't overwritten.

 I'm going to investigate further and submit a patch soon.

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


More information about the Tracker mailing list