aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Trowbridge <trow@ximian.com>2001-03-18 11:26:03 +0800
committerJon Trowbridge <trow@src.gnome.org>2001-03-18 11:26:03 +0800
commite54ed579962cf62674848a28fcf70212d7bff9ab (patch)
tree1c3cabf79300274d54eb1be73385ea5743a4c3d1
parent5908c8f777f46df1fdcdc6ac7641b7b1f6ed3c5f (diff)
downloadgsoc2013-evolution-e54ed579962cf62674848a28fcf70212d7bff9ab.tar
gsoc2013-evolution-e54ed579962cf62674848a28fcf70212d7bff9ab.tar.gz
gsoc2013-evolution-e54ed579962cf62674848a28fcf70212d7bff9ab.tar.bz2
gsoc2013-evolution-e54ed579962cf62674848a28fcf70212d7bff9ab.tar.lz
gsoc2013-evolution-e54ed579962cf62674848a28fcf70212d7bff9ab.tar.xz
gsoc2013-evolution-e54ed579962cf62674848a28fcf70212d7bff9ab.tar.zst
gsoc2013-evolution-e54ed579962cf62674848a28fcf70212d7bff9ab.zip
Start feeding query matches back to the caller before the entire search is
2001-03-17 Jon Trowbridge <trow@ximian.com> * backend/pas/pas-backend-file.c (pas_backend_file_search): Start feeding query matches back to the caller before the entire search is complete; this gives us some "instant feedback" rather than causing evolution to totally lock up... svn path=/trunk/; revision=8792
-rw-r--r--addressbook/ChangeLog7
-rw-r--r--addressbook/backend/pas/pas-backend-file.c31
2 files changed, 29 insertions, 9 deletions
diff --git a/addressbook/ChangeLog b/addressbook/ChangeLog
index f02ccf7161..8c88d4a5f7 100644
--- a/addressbook/ChangeLog
+++ b/addressbook/ChangeLog
@@ -1,5 +1,12 @@
2001-03-17 Jon Trowbridge <trow@ximian.com>
+ * backend/pas/pas-backend-file.c (pas_backend_file_search):
+ Start feeding query matches back to the caller before the
+ entire search is complete; this gives us some "instant feedback"
+ rather than causing evolution to totally lock up...
+
+2001-03-17 Jon Trowbridge <trow@ximian.com>
+
* backend/ebook/e-book.c (e_book_queue_op): Tag all queued ops.
(e_book_cancel_op): Added. Search for a pending op with a given
tag, and mark it as inactive.
diff --git a/addressbook/backend/pas/pas-backend-file.c b/addressbook/backend/pas/pas-backend-file.c
index a42a1fa499..775e322631 100644
--- a/addressbook/backend/pas/pas-backend-file.c
+++ b/addressbook/backend/pas/pas-backend-file.c
@@ -8,6 +8,7 @@
#include "config.h"
#include <gtk/gtksignal.h>
+#include <stdio.h>
#include <gnome.h>
#include <unistd.h>
#include <fcntl.h>
@@ -188,15 +189,19 @@ view_destroy(GtkObject *object, gpointer data)
PASBook *book = (PASBook *)data;
PASBackendFile *bf;
EIterator *iterator;
+ gboolean success = FALSE;
bf = PAS_BACKEND_FILE(pas_book_get_backend(book));
for (iterator = e_list_get_iterator(bf->priv->book_views); e_iterator_is_valid(iterator); e_iterator_next(iterator)) {
const PASBackendFileBookView *view = e_iterator_get(iterator);
if (view->book_view == PAS_BOOK_VIEW(object)) {
e_iterator_delete(iterator);
+ success = TRUE;
break;
}
}
+ if (!success)
+ g_warning ("Failed to remove from book_views list");
gtk_object_unref(GTK_OBJECT(iterator));
corba_book = bonobo_object_corba_objref(BONOBO_OBJECT(book));
@@ -475,10 +480,10 @@ pas_backend_file_search (PASBackendFile *bf,
{
int db_error = 0;
GList *cards = NULL;
- gint card_count = 0;
+ gint card_count = 0, card_threshold = 20, card_threshold_max = 200;
DB *db = bf->priv->file_db;
DBT id_dbt, vcard_dbt;
- int i;
+ int i, file_version_name_len;
int esexp_error;
PASBackendFileBookView *view = (PASBackendFileBookView *)cnstview;
@@ -502,6 +507,8 @@ pas_backend_file_search (PASBackendFile *bf,
e_sexp_input_text(view->search_sexp, view->search, strlen(view->search));
esexp_error = e_sexp_parse(view->search_sexp);
+ file_version_name_len = strlen (PAS_BACKEND_FILE_VERSION_NAME);
+
if (esexp_error == -1) {
pas_book_view_notify_complete (view->book_view);
return;
@@ -512,25 +519,30 @@ pas_backend_file_search (PASBackendFile *bf,
while (db_error == 0) {
/* don't include the version in the list of cards */
- if (id_dbt.size != strlen(PAS_BACKEND_FILE_VERSION_NAME) + 1
+ if (id_dbt.size != file_version_name_len+1
|| strcmp (id_dbt.data, PAS_BACKEND_FILE_VERSION_NAME)) {
char *vcard_string = vcard_dbt.data;
/* check if the vcard matches the search sexp */
if (vcard_matches_search (view, vcard_string)) {
- cards = g_list_append (cards, g_strdup (vcard_string));
+ cards = g_list_prepend (cards, g_strdup (vcard_string));
++card_count;
-#if 0
+
/* If we've accumulated a number of matches, pass them off to the client. */
- if (card_count > 50) {
+ if (card_count > card_threshold) {
pas_book_view_notify_add (view->book_view, cards);
/* Clean up the handed-off data. */
g_list_foreach (cards, (GFunc)g_free, NULL);
g_list_free (cards);
cards = NULL;
card_count = 0;
+
+ /* Yeah, this scheme is over-complicated. But I like it. */
+ if (card_threshold <= card_threshold_max) {
+ card_threshold = MIN (2*card_threshold, card_threshold_max);
+ }
}
-#endif
+
}
}
@@ -541,7 +553,7 @@ pas_backend_file_search (PASBackendFile *bf,
g_warning ("pas_backend_file_search: error building list\n");
}
- // if (card_count)
+ if (card_count)
pas_book_view_notify_add (view->book_view, cards);
pas_book_view_notify_complete (view->book_view);
@@ -982,7 +994,7 @@ pas_backend_file_process_get_book_view (PASBackend *backend,
EIterator *iterator;
g_return_if_fail (req->listener != NULL);
-
+
corba_book = bonobo_object_corba_objref(BONOBO_OBJECT(book));
CORBA_exception_init(&ev);
@@ -1018,6 +1030,7 @@ pas_backend_file_process_get_book_view (PASBackend *backend,
iterator = e_list_get_iterator(bf->priv->book_views);
e_iterator_last(iterator);
+
pas_backend_file_search (bf, book, e_iterator_get(iterator));
gtk_object_unref(GTK_OBJECT(iterator));