cli/new: replace newly deprecated n_m_has_maildir_flag
authorDavid Bremner <david@tethera.net>
Mon, 13 Jul 2020 23:51:07 +0000 (20:51 -0300)
committerDavid Bremner <david@tethera.net>
Mon, 20 Jul 2020 11:54:42 +0000 (08:54 -0300)
Boolean return values have no out-of-band-values to signal errors. The
change here is that a (somewhat unlikely) fatal error after indexing
will now be caught.

notmuch-new.c

index f079f62a63968c7ce19a2045de428864eb82e399..4075d395fd4ff89a992fbf9ed7cfbdc7a7761839 100644 (file)
@@ -388,8 +388,11 @@ add_file (notmuch_database_t *notmuch, const char *filename,
            notmuch_message_maildir_flags_to_tags (message);
 
        for (tag = state->new_tags; *tag != NULL; tag++) {
-           if (strcmp ("unread", *tag) != 0 ||
-               ! notmuch_message_has_maildir_flag (message, 'S')) {
+           notmuch_bool_t is_set;
+           /* Currently all errors from has_maildir_flag are fatal */
+           if ((status = notmuch_message_has_maildir_flag_st (message, 'S', &is_set)))
+               goto DONE;
+           if (strcmp ("unread", *tag) != 0 || ! is_set) {
                notmuch_message_add_tag (message, *tag);
            }
        }