Re: [Evolution-hackers] [CamelStore] clarifying of documentation becomes a minor change of code



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




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