[PATCH v2 2/4] new: Handle fatal errors in remove_filename and _remove_directory
Austin Clements
amdragon at MIT.EDU
Sun Apr 22 08:36:17 PDT 2012
Quoth Mark Walters on Apr 22 at 8:34 am:
> On Sun, 22 Apr 2012, Austin Clements <amdragon at MIT.EDU> wrote:
> > Previously such errors were simply ignored. Now they cause an
> > immediate cleanup and abort.
> > ---
> > notmuch-new.c | 25 +++++++++++++++++++------
> > 1 files changed, 19 insertions(+), 6 deletions(-)
> >
> > diff --git a/notmuch-new.c b/notmuch-new.c
> > index 15c0b36..92e0489 100644
> > --- a/notmuch-new.c
> > +++ b/notmuch-new.c
> > @@ -785,8 +785,10 @@ remove_filename (notmuch_database_t *notmuch,
> > add_files_state->renamed_messages++;
> > if (add_files_state->synchronize_flags == TRUE)
> > notmuch_message_maildir_flags_to_tags (message);
> > - } else
> > + status = NOTMUCH_STATUS_SUCCESS;
> > + } else if (status == NOTMUCH_STATUS_SUCCESS) {
> > add_files_state->removed_messages++;
> > + }
> > notmuch_message_destroy (message);
> > notmuch_database_end_atomic (notmuch);
> > return status;
> > @@ -794,12 +796,13 @@ remove_filename (notmuch_database_t *notmuch,
> >
> > /* Recursively remove all filenames from the database referring to
> > * 'path' (or to any of its children). */
> > -static void
> > +static notmuch_status_t
> > _remove_directory (void *ctx,
> > notmuch_database_t *notmuch,
> > const char *path,
> > add_files_state_t *add_files_state)
> > {
> > + notmuch_status_t status = NOTMUCH_STATUS_SUCCESS;
> > notmuch_directory_t *directory;
> > notmuch_filenames_t *files, *subdirs;
> > char *absolute;
> > @@ -812,8 +815,10 @@ _remove_directory (void *ctx,
> > {
> > absolute = talloc_asprintf (ctx, "%s/%s", path,
> > notmuch_filenames_get (files));
> > - remove_filename (notmuch, absolute, add_files_state);
> > + status = remove_filename (notmuch, absolute, add_files_state);
> > talloc_free (absolute);
> > + if (status)
> > + goto DONE;
> > }
> >
> > for (subdirs = notmuch_directory_get_child_directories (directory);
> > @@ -822,11 +827,15 @@ _remove_directory (void *ctx,
> > {
> > absolute = talloc_asprintf (ctx, "%s/%s", path,
> > notmuch_filenames_get (subdirs));
> > - _remove_directory (ctx, notmuch, absolute, add_files_state);
> > + status = _remove_directory (ctx, notmuch, absolute, add_files_state);
> > talloc_free (absolute);
> > + if (status)
> > + goto DONE;
> > }
> >
> > + DONE:
> > notmuch_directory_destroy (directory);
> > + return NOTMUCH_STATUS_SUCCESS;
>
> Doesn't this need to be return status or something?
Arrg. Yes, of course. I wish I knew a good way to test this stuff.
More information about the notmuch
mailing list