debian/notmuch
1
0
Fork 0
mirror of https://git.notmuchmail.org/git/notmuch synced 2024-11-22 02:48:08 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

notmuch setup/new: Propagate failure from notmuch_database_set_timestamp

Browse source 
With some recent testing, the timestamp was failing, (overflowing
the term limit), and reporting an error, but the top-level notmuch
command was still returning a success return value.

I think it's high time to add a test suite, (and the code base is
small enough that if we add it now it shouldn't be *too* hard to
shoot for a very high coverage percentage).
This commit is contained in:
Carl Worth 2009-10-24 22:11:38 -07:00
parent 0bc73af96c
commit 309c0e1cc5
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
notmuch.c
View file

@ -270,7 +270,9 @@ add_files_recursive (notmuch_database_t *notmuch,
next = NULL;
}
notmuch_database_set_timestamp (notmuch, path, path_mtime);
status = notmuch_database_set_timestamp (notmuch, path, path_mtime);
if (status && ret == NOTMUCH_STATUS_SUCCESS)
ret = status;
DONE:
if (next)