aboutsummaryrefslogtreecommitdiff
path: root/bindings/ruby/query.c
diff options
context:
space:
mode:
authorDavid Bremner <david@tethera.net>2015-09-27 12:31:59 -0300
committerDavid Bremner <david@tethera.net>2015-10-05 19:51:52 -0300
commit516e196fa12a71982ee4feb4ef6ff52f48b1805b (patch)
tree8a6fdaa98703f75bf01704d7151c76c3c8508815 /bindings/ruby/query.c
parente7b420a85446fa469df77e24c11f8afa06a2592e (diff)
ruby: use new count API
This change of replacing ignoring errors with exceptions is intended, and indeed one of the main motivations for the libnotmuch API changes.
Diffstat (limited to 'bindings/ruby/query.c')
-rw-r--r--bindings/ruby/query.c24
1 files changed, 14 insertions, 10 deletions
diff --git a/bindings/ruby/query.c b/bindings/ruby/query.c
index a7dacba3..f87700a4 100644
--- a/bindings/ruby/query.c
+++ b/bindings/ruby/query.c
@@ -173,14 +173,16 @@ VALUE
notmuch_rb_query_count_messages (VALUE self)
{
notmuch_query_t *query;
+ notmuch_status_t status;
+ unsigned int count;
Data_Get_Notmuch_Query (self, query);
- /* Xapian exceptions are not handled properly.
- * (function may return 0 after printing a message)
- * Thus there is nothing we can do here...
- */
- return UINT2NUM(notmuch_query_count_messages(query));
+ status = notmuch_query_count_messages_st (query, &count);
+ if (status)
+ notmuch_rb_status_raise (status);
+
+ return UINT2NUM(count);
}
/*
@@ -192,12 +194,14 @@ VALUE
notmuch_rb_query_count_threads (VALUE self)
{
notmuch_query_t *query;
+ notmuch_status_t status;
+ unsigned int count;
Data_Get_Notmuch_Query (self, query);
- /* Xapian exceptions are not handled properly.
- * (function may return 0 after printing a message)
- * Thus there is nothing we can do here...
- */
- return UINT2NUM(notmuch_query_count_threads(query));
+ status = notmuch_query_count_threads_st (query, &count);
+ if (status)
+ notmuch_rb_status_raise (status);
+
+ return UINT2NUM(count);
}