diff options
| author | David Bremner <david@tethera.net> | 2015-09-27 12:31:59 -0300 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2015-10-05 19:51:52 -0300 |
| commit | 516e196fa12a71982ee4feb4ef6ff52f48b1805b (patch) | |
| tree | 8a6fdaa98703f75bf01704d7151c76c3c8508815 /bindings/ruby | |
| parent | e7b420a85446fa469df77e24c11f8afa06a2592e (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')
| -rw-r--r-- | bindings/ruby/query.c | 24 |
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); } |