[Pidgin] #11262: list argument is not used in pidgin_blist_set_visible

Pidgin trac at pidgin.im
Thu Jan 28 10:05:04 EST 2010


#11262: list argument is not used in pidgin_blist_set_visible
---------------------+------------------------------------------------------
Reporter:  andrikos  |        Type:  defect      
  Status:  new       |   Component:  pidgin (gtk)
 Version:  2.6.5     |    Keywords:              
---------------------+------------------------------------------------------
 In gtkblist.c: 6804 there is the PurpleBuddyList *list argument, but
 inside the function body the gtkblist variable is used.

 This was done maybe unintentionally, but it is still a bug. I see 3
 possible fixes:
 a) rename the list argument to gtkblist. this will mask and maybe conflict
 with the global (file scope) variable gtkblist used now
 b) remove the list argument since it is not needed (if the gtkblist use
 has been done intentionally) which is the most compatible-friendly option
 c) rename the use of gtkblist in the function body to the correct list
 variable (i guess this is the correct way to go, although I am not aware
 of possible new bugs from this change)

-- 
Ticket URL: <http://developer.pidgin.im/ticket/11262>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list