debian/notmuch
1
0
Fork 0
mirror of https://git.notmuchmail.org/git/notmuch synced 2025-01-20 18:25:58 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

notmuch_message_tags_to_maildir_flags: Fix to rename multiple files

Browse source 
This function was documented as modifying every filename associated
with the message. Fix it to actually do that.
This commit is contained in:
Carl Worth 2010-11-11 03:41:39 -08:00
parent 8480a0a003
commit 71a3201885
1 changed files with 47 additions and 41 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

88
lib/message.cc
View file

@ -961,15 +961,14 @@ maildir_get_subdir (char *filename)
return subdir; return subdir;
} }
/* XXX: Needs to iterate over all filenames in the message /* XXX: Needs to ensure that existing, unsupported flags in the
*
* XXX: Needs to ensure that existing, unsupported flags in the
* filename are left unchanged (which also needs a test in the * filename are left unchanged (which also needs a test in the
* test suite). * test suite).
*/ */
notmuch_status_t notmuch_status_t
notmuch_message_tags_to_maildir_flags (notmuch_message_t *message) notmuch_message_tags_to_maildir_flags (notmuch_message_t *message)
{ {
notmuch_filenames_t *filenames;
char flags[ARRAY_SIZE(flag2tag)+1]; char flags[ARRAY_SIZE(flag2tag)+1];
const char *filename, *p; const char *filename, *p;
char *filename_new, *subdir = NULL; char *filename_new, *subdir = NULL;
@ -977,49 +976,56 @@ notmuch_message_tags_to_maildir_flags (notmuch_message_t *message)
maildir_get_new_flags (message, flags); maildir_get_new_flags (message, flags);
filename = notmuch_message_get_filename (message); for (filenames = notmuch_message_get_filenames (message);
/* TODO: Iterate over all file names. */ notmuch_filenames_valid (filenames);
p = strstr(filename, ":2,"); notmuch_filenames_move_to_next (filenames))
if ((p && strcmp (p+3, flags) == 0) || {
(!p && flags[0] == '\0')) { filename = notmuch_filenames_get (filenames);
// Return if flags are not to be changed - this suppresses
// moving the message from new/ to cur/ during initial
// tagging.
return NOTMUCH_STATUS_SUCCESS;
}
if (!p)
p = filename + strlen(filename);
filename_new = (char*)talloc_size(message, (p-filename) + 3 + sizeof(flags)); p = strstr(filename, ":2,");
if (unlikely (filename_new == NULL)) if ((p && strcmp (p+3, flags) == 0) ||
return NOTMUCH_STATUS_OUT_OF_MEMORY; (!p && flags[0] == '\0'))
{
memcpy(filename_new, filename, p-filename); continue;
filename_new[p-filename] = '\0';
/* If message is in new/ move it under cur/. */
subdir = maildir_get_subdir (filename_new);
if (subdir && memcmp (subdir, "new/", 4) == 0)
memcpy (subdir, "cur/", 4);
strcpy (filename_new+(p-filename), ":2,");
strcpy (filename_new+(p-filename)+3, flags);
if (strcmp (filename, filename_new) != 0) {
notmuch_status_t status;
ret = rename (filename, filename_new);
if (ret == -1) {
perror (talloc_asprintf (message, "rename of %s to %s failed",
filename, filename_new));
exit (1);
} }
status = _notmuch_message_rename (message, filename_new);
_notmuch_message_sync (message); if (!p)
p = filename + strlen(filename);
return status; filename_new = (char*) talloc_size (message,
(p-filename) + 3 + sizeof (flags));
if (unlikely (filename_new == NULL))
return NOTMUCH_STATUS_OUT_OF_MEMORY;
memcpy (filename_new, filename, p-filename);
filename_new[p-filename] = '\0';
/* If message is in new/ move it under cur/. */
subdir = maildir_get_subdir (filename_new);
if (subdir && memcmp (subdir, "new/", 4) == 0)
memcpy (subdir, "cur/", 4);
strcpy (filename_new+(p-filename), ":2,");
strcpy (filename_new+(p-filename)+3, flags);
if (strcmp (filename, filename_new) != 0) {
notmuch_status_t status;
ret = rename (filename, filename_new);
if (ret == -1) {
perror (talloc_asprintf (message, "rename of %s to %s failed",
filename, filename_new));
exit (1);
}
status = _notmuch_message_rename (message, filename_new);
_notmuch_message_sync (message);
if (status)
return status;
}
} }
return NOTMUCH_STATUS_SUCCESS; return NOTMUCH_STATUS_SUCCESS;
} }