Re: new API proposal find_msg_async
- From: Sergio Villar Senin <svillar igalia com>
- To: tinymail-devel-list <tinymail-devel-list gnome org>
- Subject: Re: new API proposal find_msg_async
- Date: Mon, 28 Apr 2008 17:06:55 +0200
Philip Van Hoof escribiu:
> One problem remaining:
>
> +#ifdef DBC /* require */
> + g_assert (TNY_IS_FOLDER (self));
> + g_assert (url_string);
> + g_assert (strlen (url_string) > 0);
> + g_assert (strstr (url_string, "://") != NULL);
> + g_assert (TNY_FOLDER_GET_IFACE (self)->get_msg_async!= NULL);
> +#endif
>
> I think the "://" assertion on url_string should be just ":", not "://",
> else wont maildir url_strings be allowed.
Indeed I noticed that when developing the first version. BTW Dape I'd
remove it from find_msg as well.
Br
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]