+2000-07-07 Dan Winship <danw@helixcode.com>
+
+ * camel-folder.c (camel_folder_search_by_expression): Make this
+ return a GPtrArray rather than a GList.
+
+ * camel-folder-search.c (camel_folder_search_execute_expression):
+ * providers/imap/camel-imap-folder.c (imap_search_by_expression):
+ * providers/mbox/camel-mbox-folder.c (mbox_search_by_expression):
+ * providers/nntp/camel-nntp-folder.c (nntp_search_by_expression):
+ Update to return a GPtrArray rather than a GList.
+
2000-07-07 Jeffrey Stedfast <fejj@helixcode.com>
* providers/smtp/camel-smtp-transport.c (esmtp_get_authtypes):
* @expr:
* @ex:
*
- * Execute the search expression @expr, returning a list of
- * all matches as a GList of uid's of matching messages.
+ * Execute the search expression @expr, returning an array of
+ * all matches as a GPtrArray of uid's of matching messages.
*
* Note that any settings such as set_body_index(), set_folder(),
* and so on are reset to #NULL once the search has completed.
*
- * TODO: The interface should probably return a GPtrArray
- * of summary items instead (since they are much more useful
- * to any client).
+ * TODO: The interface should probably return summary items instead
+ * (since they are much more useful to any client).
*
- * Return value: A GList of strings of all matching messages. Once
+ * Return value: A GPtrArray of strings of all matching messages. Once
* finished with this, the array AND CONTENTS should be free'd
* by the caller.
**/
-GList *
+GPtrArray *
camel_folder_search_execute_expression(CamelFolderSearch *search, const char *expr, CamelException *ex)
{
ESExpResult *r;
- GList *matches = NULL;
+ GPtrArray *matches = g_ptr_array_new ();
int i;
/* only re-parse if the search has changed */
d(printf("got result ...\n"));
for (i=0;i<r->value.ptrarray->len;i++) {
d(printf("adding match: %s\n", (char *)g_ptr_array_index(r->value.ptrarray, i)));
- matches = g_list_prepend(matches, g_strdup(g_ptr_array_index(r->value.ptrarray, i)));
+ g_ptr_array_add(matches, g_strdup(g_ptr_array_index(r->value.ptrarray, i)));
}
e_sexp_result_free(r);
} else {
void camel_folder_search_set_folder(CamelFolderSearch *search, CamelFolder *folder);
void camel_folder_search_set_summary(CamelFolderSearch *search, GPtrArray *summary);
void camel_folder_search_set_body_index(CamelFolderSearch *search, ibex *index);
-GList *camel_folder_search_execute_expression(CamelFolderSearch *search, const char *expr, CamelException *ex);
+GPtrArray *camel_folder_search_execute_expression(CamelFolderSearch *search, const char *expr, CamelException *ex);
#endif /* ! _CAMEL_FOLDER_SEARCH_H */
static const CamelMessageInfo *get_message_info (CamelFolder *folder,
const char *uid);
-static GList *search_by_expression (CamelFolder *folder, const char *exp,
- CamelException *ex);
+static GPtrArray *search_by_expression (CamelFolder *folder,
+ const char *exp,
+ CamelException *ex);
static void copy_message_to (CamelFolder *source,
const char *uid,
return folder->has_search_capability;
}
-static GList *
+static GPtrArray *
search_by_expression (CamelFolder *folder, const char *expression,
CamelException *ex)
{
* Return value: a list of uids of matching messages. The caller must
* free the list and each of the elements when it is done.
**/
-GList *
+GPtrArray *
camel_folder_search_by_expression (CamelFolder *folder, const char *expression,
CamelException *ex)
{
gboolean (*has_search_capability) (CamelFolder *folder);
- GList * (*search_by_expression) (CamelFolder *folder,
- const char *expression,
- CamelException *ex);
+ GPtrArray * (*search_by_expression) (CamelFolder *folder,
+ const char *expression,
+ CamelException *ex);
const CamelMessageInfo * (*get_message_info) (CamelFolder *,
const char *uid);
CamelException *ex);
CamelStore * camel_folder_get_parent_store (CamelFolder *folder,
CamelException *ex);
-GList * camel_folder_list_subfolders (CamelFolder *folder,
- CamelException *ex);
/* delete operations */
/* search api */
gboolean camel_folder_has_search_capability (CamelFolder *folder);
-GList * camel_folder_search_by_expression (CamelFolder *folder, const char *expression, CamelException *ex);
+GPtrArray * camel_folder_search_by_expression (CamelFolder *folder,
+ const char *expression,
+ CamelException *ex);
-/* summary info. FIXME: rename this slightly? */
+/* summary info */
const CamelMessageInfo *camel_folder_get_message_info (CamelFolder *summary,
const char *uid);
static const CamelMessageInfo *imap_get_message_info (CamelFolder *folder, const char *uid);
-static GList *imap_search_by_expression (CamelFolder *folder, const char *expression, CamelException *ex);
+static GPtrArray *imap_search_by_expression (CamelFolder *folder, const char *expression, CamelException *ex);
static void imap_finalize (GtkObject *object);
return info;
}
-static GList *
+static GPtrArray *
imap_search_by_expression (CamelFolder *folder, const char *expression, CamelException *ex)
{
return NULL;
static const CamelMessageInfo *mbox_get_message_info (CamelFolder *folder, const char *uid);
-static GList *mbox_search_by_expression(CamelFolder *folder, const char *expression, CamelException *ex);
+static GPtrArray *mbox_search_by_expression(CamelFolder *folder, const char *expression, CamelException *ex);
static guint32 mbox_get_message_flags (CamelFolder *folder, const char *uid, CamelException *ex);
static void mbox_set_message_flags (CamelFolder *folder, const char *uid, guint32 flags, guint32 set, CamelException *ex);
return camel_folder_summary_uid (CAMEL_FOLDER_SUMMARY (mbox_folder->summary), uid);
}
-static GList *
+static GPtrArray *
mbox_search_by_expression (CamelFolder *folder, const char *expression, CamelException *ex)
{
CamelMboxFolder *mbox_folder = CAMEL_MBOX_FOLDER (folder);
}
}
-static GList*
+static GPtrArray*
nntp_folder_search_by_expression (CamelFolder *folder, const char *expression, CamelException *ex)
{
g_assert (0);