[Pidgin] #14818: [PATCH] Finch should handle conversation-extended signals

Pidgin trac at pidgin.im
Sat Jan 21 16:56:01 EST 2012


#14818: [PATCH] Finch should handle conversation-extended signals
-----------------------------------------+----------------------------------
 Reporter:  hyc                          |        Owner:  sadrul             
     Type:  patch                        |       Status:  new                
Milestone:  Patches Needing Improvement  |    Component:  finch (gnt/ncurses)
  Version:  2.10.0                       |   Resolution:                     
 Keywords:  signals conversation         |  
-----------------------------------------+----------------------------------

Comment(by sadrul):

 This looks much better. Just one issue from what I can see:

 It looks like purple_menu_action_free does not free its |children|. So
 purple_menu_action_free needs to have 'g_list_free(act->children);'. In
 addition, it either also needs 'g_list_foreach(act->children,
 purple_menu_action_free, NULL);', or lines 659-660 in the patch needs to
 also be added after line 642.

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


More information about the Tracker mailing list