[Pidgin] #3778: SIMPLE subscription dialog fix

Pidgin trac at pidgin.im
Mon Nov 5 11:13:17 EST 2007


#3778: SIMPLE subscription dialog fix
-----------------------+----------------------------------------------------
  Reporter:  hawkinsw  |       Owner:  seanegan                  
      Type:  patch     |      Status:  new                       
  Priority:  minor     |   Milestone:                            
 Component:  SIMPLE    |     Version:  2.2.2                     
Resolution:            |    Keywords:  publish, subscribe, dialog
   Pending:  0         |  
-----------------------+----------------------------------------------------
Comment (by datallah):

 One thing I noticed is that in a couple places you're using free() instead
 of g_free() - this is problematic in a number of situations.

 Apart from that, it looks pretty good to me.  I don't really have a good
 way to test this, but the concept seems sound.

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


More information about the Tracker mailing list