Re: [PATCH 2/2] lib/message-property: catch xapian exceptions

2023-03-27 Thread Kevin Boulain
On 2023-03-27 at 08:30 -03, David Bremner wrote: > So you should probably test that the status is not success, and > ideally print the message. You can see some examples in > T560-lib-error.sh, in particular in the created file c_tail which > handles checking the error code and printing the

Re: [PATCH 2/2] lib/message-property: catch xapian exceptions

2023-03-27 Thread David Bremner
Kevin Boulain writes: > Since libnotmuch exposes a C interface there's no way for clients to > diff --git a/test/T610-message-property.sh b/test/T610-message-property.sh > index 944e1810..f7cabe4d 100755 > --- a/test/T610-message-property.sh > +++ b/test/T610-message-property.sh > @@ -363,7

[PATCH 2/2] lib/message-property: catch xapian exceptions

2023-02-27 Thread Kevin Boulain
Since libnotmuch exposes a C interface there's no way for clients to catch this. Inspired by what's done for tags (see notmuch_message_remove_tag). --- lib/message-property.cc | 36 +-- test/T610-message-property.sh | 4 ++-- 2 files changed, 32