On Mon, 2008-09-01 at 18:19 +0200, Paul Bolle wrote:
[snip]
>  void
>  camel_store_free_folder_info (CamelStore *store, CamelFolderInfo *fi)
>  {
> +     if (!fi)
> +             return;
> +

that null-check should probably go after the g_return below:

>       g_return_if_fail (CAMEL_IS_STORE (store));
>  
>       CS_CLASS (store)->free_folder_info (store, fi);
> 

other than that, it's ok with me.

Jeff


_______________________________________________
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers

Reply via email to