]> git.notmuchmail.org Git - notmuch/blobdiff - notmuch-new.c
notmuch new: Eliminate the check on the mtime of regular files before adding.
[notmuch] / notmuch-new.c
index 4adbdc7f60ff4280e7fe2255975afa17bf689a87..fe280d841c8f50ce2b84ff94c04770b648c32606 100644 (file)
@@ -121,20 +121,19 @@ is_maildir (struct dirent **entries, int count)
 
 /* Examine 'path' recursively as follows:
  *
- *   o Ask the filesystem for the mtime of 'path' (path_mtime)
+ *   o Ask the filesystem for the mtime of 'path' (fs_mtime)
  *
- *   o Ask the database for its timestamp of 'path' (path_dbtime)
+ *   o Ask the database for its timestamp of 'path' (db_mtime)
  *
- *   o If 'path_mtime' > 'path_dbtime'
+ *   o If 'fs_mtime' > 'db_mtime'
  *
- *       o For each regular file in 'path' with mtime newer than the
- *         'path_dbtime' call add_message to add the file to the
- *         database.
+ *       o For each regular file directly within 'path', call
+ *         add_message to add the file to the database.
  *
- *       o For each sub-directory of path, recursively call into this
- *         same function.
+ *   o For each sub-directory of path, recursively call into this
+ *     same function.
  *
- *   o Tell the database to update its time of 'path' to 'path_mtime'
+ *   o Tell the database to update its time of 'path' to 'fs_mtime'
  *
  * The 'struct stat *st' must point to a structure that has already
  * been initialized for 'path' by calling stat().
@@ -148,17 +147,17 @@ add_files_recursive (notmuch_database_t *notmuch,
     DIR *dir = NULL;
     struct dirent *entry = NULL;
     char *next = NULL;
-    time_t path_mtime, path_dbtime;
+    time_t fs_mtime, db_mtime;
     notmuch_status_t status, ret = NOTMUCH_STATUS_SUCCESS;
     notmuch_message_t *message = NULL;
     struct dirent **namelist = NULL;
     int num_entries;
     notmuch_directory_t *directory;
 
-    path_mtime = st->st_mtime;
+    fs_mtime = st->st_mtime;
 
     directory = notmuch_database_get_directory (notmuch, path);
-    path_dbtime = notmuch_directory_get_mtime (directory);
+    db_mtime = notmuch_directory_get_mtime (directory);
 
     num_entries = scandir (path, &namelist, 0, ino_cmp);
 
@@ -180,7 +179,7 @@ add_files_recursive (notmuch_database_t *notmuch,
        /* If this directory hasn't been modified since the last
         * add_files, then we only need to look further for
         * sub-directories. */
-       if (path_mtime <= path_dbtime && entry->d_type == DT_REG)
+       if (fs_mtime <= db_mtime && entry->d_type == DT_REG)
            continue;
 
        /* Ignore special directories to avoid infinite recursion.
@@ -219,66 +218,62 @@ add_files_recursive (notmuch_database_t *notmuch,
        }
 
        if (S_ISREG (st->st_mode)) {
-           /* If the file hasn't been modified since the last
-            * add_files, then we need not look at it. */
-           if (path_dbtime == 0 || st->st_mtime > path_dbtime) {
-               state->processed_files++;
-
-               if (state->verbose) {
-                   if (state->output_is_a_tty)
-                       printf("\r\033[K");
-
-                   printf ("%i/%i: %s",
-                           state->processed_files,
-                           state->total_files,
-                           next);
-
-                   putchar((state->output_is_a_tty) ? '\r' : '\n');
-                   fflush (stdout);
-               }
-
-               status = notmuch_database_add_message (notmuch, next, &message);
-               switch (status) {
-                   /* success */
-                   case NOTMUCH_STATUS_SUCCESS:
-                       state->added_messages++;
-                       tag_inbox_and_unread (message);
-                       break;
-                   /* Non-fatal issues (go on to next file) */
-                   case NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID:
-                       /* Stay silent on this one. */
-                       break;
-                   case NOTMUCH_STATUS_FILE_NOT_EMAIL:
-                       fprintf (stderr, "Note: Ignoring non-mail file: %s\n",
-                                next);
-                       break;
-                   /* Fatal issues. Don't process anymore. */
-                   case NOTMUCH_STATUS_READONLY_DATABASE:
-                   case NOTMUCH_STATUS_XAPIAN_EXCEPTION:
-                   case NOTMUCH_STATUS_OUT_OF_MEMORY:
-                       fprintf (stderr, "Error: %s. Halting processing.\n",
-                                notmuch_status_to_string (status));
-                       ret = status;
-                       goto DONE;
-                   default:
-                   case NOTMUCH_STATUS_FILE_ERROR:
-                   case NOTMUCH_STATUS_NULL_POINTER:
-                   case NOTMUCH_STATUS_TAG_TOO_LONG:
-                   case NOTMUCH_STATUS_UNBALANCED_FREEZE_THAW:
-                   case NOTMUCH_STATUS_LAST_STATUS:
-                       INTERNAL_ERROR ("add_message returned unexpected value: %d",  status);
-                       goto DONE;
-               }
-
-               if (message) {
-                   notmuch_message_destroy (message);
-                   message = NULL;
-               }
-
-               if (do_add_files_print_progress) {
-                   do_add_files_print_progress = 0;
-                   add_files_print_progress (state);
-               }
+           state->processed_files++;
+
+           if (state->verbose) {
+               if (state->output_is_a_tty)
+                   printf("\r\033[K");
+
+               printf ("%i/%i: %s",
+                       state->processed_files,
+                       state->total_files,
+                       next);
+
+               putchar((state->output_is_a_tty) ? '\r' : '\n');
+               fflush (stdout);
+           }
+
+           status = notmuch_database_add_message (notmuch, next, &message);
+           switch (status) {
+           /* success */
+           case NOTMUCH_STATUS_SUCCESS:
+               state->added_messages++;
+               tag_inbox_and_unread (message);
+               break;
+           /* Non-fatal issues (go on to next file) */
+           case NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID:
+               /* Stay silent on this one. */
+               break;
+           case NOTMUCH_STATUS_FILE_NOT_EMAIL:
+               fprintf (stderr, "Note: Ignoring non-mail file: %s\n",
+                        next);
+               break;
+           /* Fatal issues. Don't process anymore. */
+           case NOTMUCH_STATUS_READONLY_DATABASE:
+           case NOTMUCH_STATUS_XAPIAN_EXCEPTION:
+           case NOTMUCH_STATUS_OUT_OF_MEMORY:
+               fprintf (stderr, "Error: %s. Halting processing.\n",
+                        notmuch_status_to_string (status));
+               ret = status;
+               goto DONE;
+           default:
+           case NOTMUCH_STATUS_FILE_ERROR:
+           case NOTMUCH_STATUS_NULL_POINTER:
+           case NOTMUCH_STATUS_TAG_TOO_LONG:
+           case NOTMUCH_STATUS_UNBALANCED_FREEZE_THAW:
+           case NOTMUCH_STATUS_LAST_STATUS:
+               INTERNAL_ERROR ("add_message returned unexpected value: %d",  status);
+               goto DONE;
+           }
+
+           if (message) {
+               notmuch_message_destroy (message);
+               message = NULL;
+           }
+
+           if (do_add_files_print_progress) {
+               do_add_files_print_progress = 0;
+               add_files_print_progress (state);
            }
        } else if (S_ISDIR (st->st_mode)) {
            status = add_files_recursive (notmuch, next, st, state);
@@ -291,7 +286,7 @@ add_files_recursive (notmuch_database_t *notmuch,
     }
 
     if (! interrupted) {
-       status = notmuch_directory_set_mtime (directory, path_mtime);
+       status = notmuch_directory_set_mtime (directory, fs_mtime);
        if (status && ret == NOTMUCH_STATUS_SUCCESS)
            ret = status;
     }