[Pidgin] #192: document prpl_info->list_icon(NULL, NULL)

[Pidgin]

#192: document prpl_info->list_icon(NULL, NULL)
------------------------+---------------------------------------------------
  Reporter:  jeff       |       Owner:                         
      Type:  patch      |      Status:  reopened               
  Priority:  minor      |   Milestone:                         
 Component:  libpurple  |     Version:  2.0                    
Resolution:             |    Keywords:  msimprpl msim myspaceim
------------------------+---------------------------------------------------
Changes (by jeff):

 * cc: seanegan at gmail.com (added)

Comment:

 Still getting up to speed with mtn, but I committed the patch to my local
 database. Will commit to pidgin.im once my [wiki:jeff public key] is added
 to pidgin.im's database.

 I'm not clear on your concern about `g_return_val_if_fail()`. I use this
 function to validate the parameters of my functions. For example:

 {{{
 static void msim_connect_cb(gpointer data, gint source, const gchar
 *error_message)
 {
     g_return_if_fail(data != NULL);
 }}}

 Here `data` should never be NULL, so I check it with
 `g_return_val_if_fail()`. This is correct usage, right?

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