diff options
| author | David Bremner <david@tethera.net> | 2022-05-23 20:39:00 -0300 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2022-06-25 12:55:02 -0300 |
| commit | 879ec9d76ad5caeb728c3f047d1da425d0b511eb (patch) | |
| tree | c5800193764e2c7e352725c4bc382013c603aec5 /lib/message.cc | |
| parent | b102d0ad11a3b4c605f6561fcfa93805b0e7d712 (diff) | |
lib/message: check return status from _n_m_add_{path,folder}_terms
Mainly to propagate information about Xapian exceptions.
Diffstat (limited to 'lib/message.cc')
| -rw-r--r-- | lib/message.cc | 23 |
1 files changed, 17 insertions, 6 deletions
diff --git a/lib/message.cc b/lib/message.cc index b04efbea..e2069123 100644 --- a/lib/message.cc +++ b/lib/message.cc @@ -792,7 +792,7 @@ is_maildir (const char *p) } /* Add "folder:" term for directory. */ -static notmuch_status_t +NODISCARD static notmuch_status_t _notmuch_message_add_folder_terms (notmuch_message_t *message, const char *directory) { @@ -842,7 +842,7 @@ _notmuch_message_add_folder_terms (notmuch_message_t *message, #define RECURSIVE_SUFFIX "/**" /* Add "path:" terms for directory. */ -static notmuch_status_t +NODISCARD static notmuch_status_t _notmuch_message_add_path_terms (notmuch_message_t *message, const char *directory) { @@ -898,6 +898,7 @@ _notmuch_message_add_directory_terms (void *ctx, notmuch_message_t *message) const char *direntry, *directory; char *colon; const std::string &term = *i; + notmuch_status_t term_status; /* Terminate loop at first term without desired prefix. */ if (strncmp (term.c_str (), direntry_prefix, direntry_prefix_len)) @@ -918,8 +919,13 @@ _notmuch_message_add_directory_terms (void *ctx, notmuch_message_t *message) message->notmuch, directory_id); - _notmuch_message_add_folder_terms (message, directory); - _notmuch_message_add_path_terms (message, directory); + term_status = _notmuch_message_add_folder_terms (message, directory); + if (term_status) + return term_status; + + term_status = _notmuch_message_add_path_terms (message, directory); + if (term_status) + return term_status; } return status; @@ -963,8 +969,13 @@ _notmuch_message_add_filename (notmuch_message_t *message, if (status) return status; - _notmuch_message_add_folder_terms (message, directory); - _notmuch_message_add_path_terms (message, directory); + status = _notmuch_message_add_folder_terms (message, directory); + if (status) + return status; + + status = _notmuch_message_add_path_terms (message, directory); + if (status) + return status; talloc_free (local); |