diff options
author | Michael Catanzaro <mcatanzaro@igalia.com> | 2016-02-06 14:42:16 -0600 |
---|---|---|
committer | Michael Catanzaro <mcatanzaro@igalia.com> | 2016-02-06 14:44:46 -0600 |
commit | 28a8208c9106c9d3d7fdde810620dbb085b2ce70 (patch) | |
tree | 44eebbf6fd68c009fa8a42e8b86b0dedbed57258 | |
parent | 3af9ee7faa02294ea8717e290f21fc86f3eb22d5 (diff) | |
download | yelp-28a8208c9106c9d3d7fdde810620dbb085b2ce70.tar.gz |
yelp-document: Fix return type of document_indexed
It's a GSourceFunc, so it'd better return a gboolean. This doesn't seem
to be causing any problems in Fedora, but in Endless it's causing 100%
CPU usage after executing a search, because the source never gets
removed.
https://bugzilla.gnome.org/show_bug.cgi?id=761647
-rw-r--r-- | libyelp/yelp-document.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/libyelp/yelp-document.c b/libyelp/yelp-document.c index c7c2f1f1..f26ebd2b 100644 --- a/libyelp/yelp-document.c +++ b/libyelp/yelp-document.c @@ -114,7 +114,7 @@ static gboolean document_request_page (YelpDocument *document, GCancellable *cancellable, YelpDocumentCallback callback, gpointer user_data); -static void document_indexed (YelpDocument *document); +static gboolean document_indexed (YelpDocument *document); static const gchar * document_read_contents (YelpDocument *document, const gchar *page_id); static void document_finish_read (YelpDocument *document, @@ -786,7 +786,7 @@ yelp_document_set_page_icon (YelpDocument *document, g_mutex_unlock (&document->priv->mutex); } -static void +static gboolean document_indexed (YelpDocument *document) { g_mutex_lock (&document->priv->mutex); @@ -799,6 +799,8 @@ document_indexed (YelpDocument *document) document->priv->reqs_search); } g_mutex_unlock (&document->priv->mutex); + + return FALSE; } /******************************************************************************/ |