Re: [GnomeMeeting-devel-list] Libebook issues



> Damien Sandras a écrit :
>>>Which makes me think the code probably should be :
>>>GError *error = NULL;
>>>
>>>ebook = e_book_new_default_addressbook (&error);
>>>
>>>if (error) {
>>>
>>>   g_error_free (error); /* don't leak ! */
>>>   return NULL;
>>>}
>
>> It will just print a message in the terminal, that's not appropriate for
>> an UI. I wouldn't do it.
>
> Notice that I used g_print for debugging purposes, but the proposed new
> code doesn't print anything ;-)

Right, so it serves no purpose. You can pass NULL when you don't have a
GError handler.


>
> Snark
> _______________________________________________
> Gnomemeeting-devel-list mailing list
> Gnomemeeting-devel-list gnome org
> http://mail.gnome.org/mailman/listinfo/gnomemeeting-devel-list
>





[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]