Fixed a bug for the last mail of each db (it was ignored ...) + redefined "today"
[mymail.git] / mymail.c
index 8926cd5..5e2cb21 100644 (file)
--- a/mymail.c
+++ b/mymail.c
@@ -46,7 +46,7 @@
 #include <time.h>
 
 #define MYMAIL_DB_MAGIC_TOKEN "mymail_index_file"
-#define VERSION "0.9.2"
+#define VERSION "0.9.5"
 
 #define MAX_NB_SEARCH_CONDITIONS 32
 
 
 #define LEADING_FROM_LINE_REGEXP_STRING "^From [^ ]*  \\(Mon\\|Tue\\|Wed\\|Thu\\|Fri\\|Sat\\|Sun\\) \\(Jan\\|Feb\\|Mar\\|Apr\\|May\\|Jun\\|Jul\\|Aug\\|Sep\\|Oct\\|Nov\\|Dec\\) [ 123][0-9] [0-9][0-9]:[0-9][0-9]:[0-9][0-9] [0-9][0-9][0-9][0-9]\n$"
 
-regex_t leading_from_line_regexp;
-
 /* Global variables! */
 
-int paranoid;
-int quiet;
-int ignore_dot_files;
+int global_quiet;
+int global_use_leading_time;
+
+regex_t global_leading_from_line_regexp;
 
 /********************************************************************/
 
@@ -78,7 +77,7 @@ enum {
   MAX_ID
 };
 
-static char *field_names[] = {
+static char *field_keys[] = {
   "mail",
   "lead",
   "from",
@@ -155,11 +154,12 @@ int xor(int a, int b) {
 const char *parse_token(char *token_buffer, size_t token_buffer_size,
                         char separator, const char *string) {
   char *u = token_buffer;
+  while(*string == separator) { string++; }
   while(u < token_buffer + token_buffer_size - 1 && *string &&
         *string != separator) {
     *(u++) = *(string++);
   }
-  while(*string == separator) string++;
+  while(*string == separator) { string++; }
   *u = '\0';
   return string;
 }
@@ -183,7 +183,6 @@ char *default_value(char *current_value,
 
 FILE *safe_fopen(const char *path, const char *mode, const char *comment) {
   FILE *result = fopen(path, mode);
-  /* printf("Opening '%s' with mode \"%s\"\n", path, mode); */
   if(result) {
     return result;
   } else {
@@ -212,6 +211,9 @@ void print_usage(FILE *out) {
   fprintf(out, "         print the version number\n");
   fprintf(out, " -q, --quiet\n");
   fprintf(out, "         do not print information during search\n");
+  fprintf(out, " -t, --use-leading-time\n");
+  fprintf(out, "         use the time stamp from the leading line of each mail and not the Date:\n");
+  fprintf(out, "         field\n");
   fprintf(out, " -p <db filename pattern>, --db-pattern <db filename pattern>\n");
   fprintf(out, "         set the db filename pattern for recursive search\n");
   fprintf(out, " -r <db root path>, --db-root <db root path>\n");
@@ -234,50 +236,50 @@ void print_usage(FILE *out) {
 
 /*********************************************************************/
 
-time_t time_for_past_day(int day) {
-  time_t t;
-  struct tm *tm;
-  int delta_day;
-  t = time(0);
-  tm = localtime(&t);
-  delta_day = (7 + tm->tm_wday - day) % 7 + 1;
-  return t - delta_day * 3600 * 24 + tm->tm_sec + 60 * tm->tm_min + 3600 * tm->tm_hour;
-}
-
-/*********************************************************************/
-
 int ignore_entry(const char *name) {
   return
     strcmp(name, ".") == 0 ||
     strcmp(name, "..") == 0 ||
-    (ignore_dot_files && name[0] == '.' && name[1] != '/');
+    (name[0] == '.' && name[1] != '/');
 }
 
 int is_a_leading_from_line(char *mbox_line) {
   return
     strncmp(mbox_line, "From ", 5) == 0 &&
-    regexec(&leading_from_line_regexp, mbox_line, 0, 0, 0) == 0;
+    regexec(&global_leading_from_line_regexp, mbox_line, 0, 0, 0) == 0;
 }
 
 int mbox_line_match_search(struct search_condition *condition,
                            int mbox_id, const char *mbox_value) {
 
   if(condition->field_id == ID_INTERVAL) {
-    if(mbox_id == ID_LEADING_LINE) {
-      const char *c;
-      time_t t;
-      struct tm tm;
-
-      c = mbox_value;
-      while(*c && *c != ' ') c++; while(*c && *c == ' ') c++;
-      strptime(c, "%a %b %e %k:%M:%S %Y", &tm);
-      t = mktime(&tm);
-
-      return (t >= condition->interval_start &&
-              (condition->interval_stop == 0 ||
-               t <= condition->interval_stop));
+    const char *c;
+    time_t t;
+    struct tm tm;
+    if(global_use_leading_time) {
+      if(mbox_id == ID_LEADING_LINE) {
+        c = mbox_value;
+        while(*c && *c != ' ') c++; while(*c && *c == ' ') c++;
+        strptime(c, "%a %b %e %k:%M:%S %Y", &tm);
+        t = mktime(&tm);
+        return (t >= condition->interval_start &&
+                (condition->interval_stop == 0 ||
+                 t <= condition->interval_stop));
+      } else {
+        return 0;
+      }
     } else {
-      return 0;
+      if(mbox_id == ID_DATE) {
+        if(!strptime(mbox_value, "%a, %d %b %Y %k:%M:%S", &tm)) {
+          strptime(mbox_value, "%d %b %Y %k:%M:%S", &tm);
+        }
+        t = mktime(&tm);
+        return (t >= condition->interval_start &&
+                (condition->interval_stop == 0 ||
+                 t <= condition->interval_stop));
+      } else {
+        return 0;
+      }
     }
   } else {
     return
@@ -368,6 +370,45 @@ void extract_mail(const char *mail_filename, unsigned long int position_in_mail,
   fclose(mail_file);
 }
 
+int check_full_mail_match(char *current_mail_filename,
+                          int nb_search_conditions,
+                          struct search_condition *search_conditions,
+                          int nb_body_conditions,
+                          int *hits,
+                          int current_position_in_mail) {
+  int n, nb_fulfilled_body_conditions;
+
+  /* We first check all conditions but the body ones */
+
+  for(n = 0; n < nb_search_conditions &&
+        ((search_conditions[n].field_id == ID_BODY) ||
+         xor(hits[n], search_conditions[n].negation)); n++);
+
+  if(n == nb_search_conditions) {
+
+    /* Now check the body ones */
+
+    nb_fulfilled_body_conditions = 0;
+
+    if(nb_body_conditions > 0) {
+      update_body_hits(current_mail_filename, current_position_in_mail,
+                       nb_search_conditions, search_conditions,
+                       nb_body_conditions,
+                       hits);
+
+      for(n = 0; n < nb_search_conditions; n++) {
+        if(search_conditions[n].field_id == ID_BODY &&
+           xor(hits[n], search_conditions[n].negation)) {
+          nb_fulfilled_body_conditions++;
+        }
+      }
+    }
+    return nb_body_conditions == nb_fulfilled_body_conditions;
+  } else {
+    return 0;
+  }
+}
+
 int search_in_db(const char *db_filename,
                  int nb_search_conditions,
                  struct search_condition *search_conditions,
@@ -377,17 +418,17 @@ int search_in_db(const char *db_filename,
   char raw_db_line[BUFFER_SIZE];
   char current_mail_filename[PATH_MAX + 1];
   unsigned long int current_position_in_mail;
-  char mbox_name[TOKEN_BUFFER_SIZE];
+  char mbox_key[TOKEN_BUFFER_SIZE];
   const char *mbox_value;
   int mbox_id;
   int already_written, m, n;
-  int nb_body_conditions, nb_fulfilled_body_conditions;
+  int nb_body_conditions;
   FILE *db_file;
   int nb_extracted_mails;
 
   nb_extracted_mails = 0;
 
-  if(!quiet) {
+  if(!global_quiet) {
     printf("Searching in '%s' ... ", db_filename);
     fflush(stdout);
   }
@@ -427,48 +468,18 @@ int search_in_db(const char *db_filename,
   strcpy(current_mail_filename, "");
 
   while(fgets(raw_db_line, BUFFER_SIZE, db_file)) {
-    mbox_value = parse_token(mbox_name, TOKEN_BUFFER_SIZE, ' ', raw_db_line);
+    mbox_value = parse_token(mbox_key, TOKEN_BUFFER_SIZE, ' ', raw_db_line);
 
-    if(strcmp("mail", mbox_name) == 0) {
+    if(strcmp("mail", mbox_key) == 0) {
       char position_in_file_string[TOKEN_BUFFER_SIZE];
-
-      if(current_mail_filename[0]) {
-
-        /* We first check all conditions but the body ones */
-
-        for(n = 0; n < nb_search_conditions &&
-              ((search_conditions[n].field_id == ID_BODY) ||
-               xor(hits[n], search_conditions[n].negation)); n++);
-
-        if(n == nb_search_conditions) {
-
-          /* Now check the body ones */
-
-          if(nb_body_conditions > 0) {
-            update_body_hits(current_mail_filename, current_position_in_mail,
-                             nb_search_conditions, search_conditions,
-                             nb_body_conditions,
-                             hits);
-          }
-
-          nb_fulfilled_body_conditions = 0;
-
-          for(n = 0; n < nb_search_conditions; n++) {
-            if(search_conditions[n].field_id == ID_BODY &&
-               xor(hits[n], search_conditions[n].negation)) {
-              nb_fulfilled_body_conditions++;
-            }
-          }
-
-          if(nb_body_conditions == nb_fulfilled_body_conditions) {
-            nb_extracted_mails++;
-            extract_mail(current_mail_filename, current_position_in_mail, output_file);
-          }
-        }
+      if(current_mail_filename[0] &&
+         check_full_mail_match(current_mail_filename,
+                               nb_search_conditions, search_conditions,
+                               nb_body_conditions, hits, current_position_in_mail)) {
+        nb_extracted_mails++;
+        extract_mail(current_mail_filename, current_position_in_mail, output_file);
       }
-
       for(n = 0; n < nb_search_conditions; n++) { hits[n] = 0; }
-
       mbox_value = parse_token(position_in_file_string, TOKEN_BUFFER_SIZE, ' ', mbox_value);
       mbox_value = parse_token(current_mail_filename, TOKEN_BUFFER_SIZE, '\n', mbox_value);
       current_position_in_mail = atol(position_in_file_string);
@@ -478,7 +489,7 @@ int search_in_db(const char *db_filename,
     else {
       mbox_id = -1;
       for(m = 0; (m < MAX_ID) && mbox_id == -1; m++) {
-        if(strncmp(field_names[m], mbox_name, strlen(mbox_name)) == 0) {
+        if(strncmp(field_keys[m], mbox_key, strlen(mbox_key)) == 0) {
           mbox_id = m;
         }
       }
@@ -489,9 +500,17 @@ int search_in_db(const char *db_filename,
     }
   }
 
+  if(current_mail_filename[0] &&
+     check_full_mail_match(current_mail_filename,
+                           nb_search_conditions, search_conditions,
+                           nb_body_conditions, hits, current_position_in_mail)) {
+    nb_extracted_mails++;
+    extract_mail(current_mail_filename, current_position_in_mail, output_file);
+  }
+
   fclose(db_file);
 
-  if(!quiet) {
+  if(!global_quiet) {
     printf("done.\n");
     fflush(stdout);
   }
@@ -548,14 +567,15 @@ int recursive_search_in_db(const char *entry_name, regex_t *db_filename_regexp,
 
 /*********************************************************************/
 
-void index_one_mbox_line(int nb_fields_to_parse, struct parsable_field *fields_to_parse,
+void index_one_mbox_line(unsigned int nb_fields_to_parse,
+                         struct parsable_field *fields_to_parse,
                          char *raw_mbox_line, FILE *db_file) {
   regmatch_t matches;
-  int f;
+  unsigned int f;
   for(f = 0; f < nb_fields_to_parse; f++) {
     if(regexec(&fields_to_parse[f].regexp, raw_mbox_line, 1, &matches, 0) == 0) {
       fprintf(db_file, "%s %s\n",
-              field_names[fields_to_parse[f].id],
+              field_keys[fields_to_parse[f].id],
               raw_mbox_line + matches.rm_eo);
     }
   }
@@ -581,17 +601,29 @@ void index_mbox(const char *mbox_filename,
 
   while(fgets(raw_mbox_line, BUFFER_SIZE, file)) {
     if(is_a_leading_from_line(raw_mbox_line)) {
+      /* This starts a new mail */
       if(in_header) {
         fprintf(stderr,
                 "Got a ^\"From \" in the header in %s:%lu.\n",
                 mbox_filename, position_in_file);
         fprintf(stderr, "%s", raw_mbox_line);
-        if(paranoid) { exit(EXIT_FAILURE); }
       }
+
+      /* printf("LEADING_LINE %s", raw_mbox_line); */
+
       in_header = 1;
       new_header = 1;
     } else if(raw_mbox_line[0] == '\n') {
-      if(in_header) { in_header = 0; }
+      if(in_header) {
+        in_header = 0;
+        /* We leave the header, index the current line */
+        if(full_line[0]) {
+          /* printf("INDEX %s\n", full_line); */
+          index_one_mbox_line(nb_fields_to_parse, fields_to_parse, full_line, db_file);
+        }
+        end_of_full_line = full_line;
+        *end_of_full_line = '\0';
+      }
     }
 
     if(in_header) {
@@ -601,6 +633,7 @@ void index_mbox(const char *mbox_filename,
       }
 
       if(raw_mbox_line[0] == ' ' || raw_mbox_line[0] == '\t') {
+        /* Continuation of a line */
         char *start = raw_mbox_line;
         while(*start == ' ' || *start == '\t') start++;
         *(end_of_full_line++) = ' ';
@@ -612,17 +645,10 @@ void index_mbox(const char *mbox_filename,
       }
 
       else {
-        /*
-          if(!((raw_mbox_line[0] >= 'a' && raw_mbox_line[0] <= 'z') ||
-          (raw_mbox_line[0] >= 'A' && raw_mbox_line[0] <= 'Z'))) {
-          fprintf(stderr,
-          "Header line syntax error %s:%lu.\n",
-          mbox_filename, position_in_file);
-          fprintf(stderr, "%s", raw_mbox_line);
-          }
-        */
+        /* Start a new header line, not a continuation */
 
         if(full_line[0]) {
+          /* printf("INDEX %s\n", full_line); */
           index_one_mbox_line(nb_fields_to_parse, fields_to_parse, full_line, db_file);
         }
 
@@ -690,6 +716,7 @@ static struct option long_options[] = {
   { "help", no_argument, 0, 'h' },
   { "version", no_argument, 0, 'v' },
   { "quiet", no_argument, 0, 'q' },
+  { "use-leading-time", no_argument, 0, 't' },
   { "db-file-generate", 1, 0, 'd' },
   { "db-pattern", 1, 0, 'p' },
   { "db-root", 1, 0, 'r' },
@@ -702,12 +729,56 @@ static struct option long_options[] = {
   { 0, 0, 0, 0 }
 };
 
+struct time_criterion {
+  char *label;
+  int start_hour, end_hour;
+  int past_week_day;
+};
+
 /*********************************************************************/
 
+static struct time_criterion time_criteria[] = {
+
+  { "8h",         8,       -1, -1 },
+  { "24h",       24,       -1, -1 },
+  { "48h",       48,       -1, -1 },
+  { "week",      24 *   7, -1, -1 },
+  { "month",     24 *  31, -1, -1 },
+  { "year",      24 * 365, -1, -1 },
+
+  { "yesterday", 48,       24, -1 },
+
+  { "today",     -1,       -1,  0 },
+  { "monday",    -1,       -1,  1 },
+  { "tuesday",   -1,       -1,  2 },
+  { "wednesday", -1,       -1,  3 },
+  { "thursday",  -1,       -1,  4 },
+  { "friday",    -1,       -1,  5 },
+  { "saturday",  -1,       -1,  6 },
+  { "sunday",    -1,       -1,  7 },
+
+};
+
+/*********************************************************************/
+
+time_t time_for_past_day(int day) {
+  time_t t;
+  struct tm *tm;
+  int delta_day;
+  t = time(0);
+  tm = localtime(&t);
+  if(day > 0) {
+    delta_day = (7 + tm->tm_wday - day) % 7;
+  } else {
+    delta_day = 0;
+  }
+  return t - (delta_day * 3600 * 24 + tm->tm_sec + 60 * tm->tm_min + 3600 * tm->tm_hour);
+}
+
 void init_condition(struct search_condition *condition, const char *full_string,
                     const char *default_search_field) {
   char full_search_field[TOKEN_BUFFER_SIZE], *search_field;
-  int m;
+  unsigned int k, m;
   const char *string;
 
   string = parse_token(full_search_field, TOKEN_BUFFER_SIZE, ' ', full_string);
@@ -720,118 +791,56 @@ void init_condition(struct search_condition *condition, const char *full_string,
     condition->negation = 0;
   }
 
-  /* Recently */
+  condition->field_id = -1;
 
-  if(strcmp(search_field, "8h") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time(0) - 3600 * 8;
-    condition->interval_stop = 0;
-  }
-
-  else if(strcmp(search_field, "24h") == 0 ||
-          strcmp(search_field, "today") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time(0) - 3600 * 24;
-    condition->interval_stop = 0;
-  }
-
-  else if(strcmp(search_field, "week") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time(0) - 3600 * 24 * 7;
-    condition->interval_stop = 0;
-  }
-
-  else if(strcmp(search_field, "month") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time(0) - 3600 * 24 * 31;
-    condition->interval_stop = 0;
-  }
-
-  else if(strcmp(search_field, "year") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time(0) - 3600 * 24 * 365;
-    condition->interval_stop = 0;
-  }
-
-  /* Yesterday */
-
-  else if(strcmp(search_field, "yesterday") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time(0) - 2 * 3600 * 24;
-    condition->interval_stop = condition->interval_start + 3600 * 24;
-  }
-
-  /* Week days */
-
-  else if(strcmp(search_field, "monday") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time_for_past_day(1);
-    condition->interval_stop = condition->interval_start + 3600 * 24;
-  }
-
-  else if(strcmp(search_field, "tuesday") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time_for_past_day(2);
-    condition->interval_stop = condition->interval_start + 3600 * 24;
-  }
-
-  else if(strcmp(search_field, "wednesday") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time_for_past_day(3);
-    condition->interval_stop = condition->interval_start + 3600 * 24;
-  }
-
-  else if(strcmp(search_field, "thursday") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time_for_past_day(4);
-    condition->interval_stop = condition->interval_start + 3600 * 24;
-  }
-
-  else if(strcmp(search_field, "friday") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time_for_past_day(5);
-    condition->interval_stop = condition->interval_start + 3600 * 24;
-  }
-
-  else if(strcmp(search_field, "saturday") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time_for_past_day(6);
-    condition->interval_stop = condition->interval_start + 3600 * 24;
-  }
+  /* Recently */
 
-  else if(strcmp(search_field, "sunday") == 0) {
-    condition->field_id = ID_INTERVAL;
-    condition->interval_start = time_for_past_day(7);
-    condition->interval_stop = condition->interval_start + 3600 * 24;
+  for(k = 0; k < sizeof(time_criteria) / sizeof(struct time_criterion); k++) {
+    if(strcmp(time_criteria[k].label, search_field) == 0) {
+      condition->field_id = ID_INTERVAL;
+      if(time_criteria[k].past_week_day < 0) {
+        condition->interval_start = time(0) - 3600 * time_criteria[k].start_hour;
+        if(time_criteria[k].end_hour >= 0) {
+          condition->interval_stop = time(0) - 3600 * time_criteria[k].end_hour;
+        } else {
+          condition->interval_stop = 0;
+        }
+      } else {
+        condition->interval_start = time_for_past_day(time_criteria[k].past_week_day);
+        condition->interval_stop = condition->interval_start + 3600 * 24;
+      }
+      break;
+    }
   }
 
-  else {
-
-    /* header-related conditions */
+  if(condition->field_id == -1) {
 
-    condition->field_id = -1;
+    /* No time condition matched, look for the search fields */
 
     for(m = 0; (m < MAX_ID) && condition->field_id == -1; m++) {
-      if(strncmp(field_names[m], search_field, strlen(search_field)) == 0) {
+      if(strncmp(field_keys[m], search_field, strlen(search_field)) == 0) {
         condition->field_id = m;
       }
     }
 
+    /* None match, if there is a default search field, re-run the search with it */
+
     if(condition->field_id == -1) {
       if(default_search_field) {
         for(m = 0; (m < MAX_ID) && condition->field_id == -1; m++) {
-          if(strncmp(field_names[m],
+          if(strncmp(field_keys[m],
                      default_search_field, strlen(default_search_field)) == 0) {
             condition->field_id = m;
           }
         }
         string = full_string;
+        if(string[0] == '!') { string++; }
       }
     }
 
     if(condition->field_id == -1) {
       fprintf(stderr,
-              "mymail: Syntax error in field name \"%s\".\n",
+              "mymail: Syntax error in field key \"%s\".\n",
               search_field);
       exit(EXIT_FAILURE);
     }
@@ -842,7 +851,7 @@ void init_condition(struct search_condition *condition, const char *full_string,
       fprintf(stderr,
               "mymail: Syntax error in regexp \"%s\" for field \"%s\".\n",
               string,
-              field_names[condition->field_id]);
+              field_keys[condition->field_id]);
       exit(EXIT_FAILURE);
     }
   }
@@ -868,22 +877,22 @@ int main(int argc, char **argv) {
   char output_filename[PATH_MAX + 1];
   int action_index = 0;
   int error = 0, show_help = 0;
-  const int nb_fields_to_parse = sizeof(fields_to_parse) / sizeof(struct parsable_field);
+  const unsigned int nb_fields_to_parse =
+    sizeof(fields_to_parse) / sizeof(struct parsable_field);
   char c;
-  int f, n;
-  int nb_search_conditions;
+  unsigned int f, n;
+  unsigned int nb_search_conditions;
   struct search_condition search_conditions[MAX_NB_SEARCH_CONDITIONS];
 
-  if(regcomp(&leading_from_line_regexp, LEADING_FROM_LINE_REGEXP_STRING, 0)) {
+  if(regcomp(&global_leading_from_line_regexp, LEADING_FROM_LINE_REGEXP_STRING, 0)) {
     fprintf(stderr,
             "mymail: Cannot compile leading \"from\" line regexp. That is strange.\n");
     exit(EXIT_FAILURE);
   }
 
-  paranoid = 0;
-  quiet = 0;
+  global_quiet = 0;
+  global_use_leading_time = 0;
   default_search_field = 0;
-  ignore_dot_files = 1;
   strncpy(output_filename, "", PATH_MAX);
 
   setlocale(LC_ALL, "");
@@ -904,7 +913,11 @@ int main(int argc, char **argv) {
       break;
 
     case 'q':
-      quiet = 1;
+      global_quiet = 1;
+      break;
+
+    case 't':
+      global_use_leading_time = 1;
       break;
 
     case 'i':
@@ -983,7 +996,7 @@ int main(int argc, char **argv) {
 
   db_filename_regexp_string = default_value(db_filename_regexp_string,
                                             "MYMAIL_DB_FILE",
-                                            "\.db$");
+                                            "\\.db$");
 
   db_root_path = default_value(db_root_path,
                                "MYMAIL_DB_ROOT",
@@ -1039,7 +1052,7 @@ int main(int argc, char **argv) {
         fprintf(stderr,
                 "mymail: Syntax error in regexp \"%s\" for field \"%s\".\n",
                 fields_to_parse[f].regexp_string,
-                field_names[fields_to_parse[f].id]);
+                field_keys[fields_to_parse[f].id]);
         exit(EXIT_FAILURE);
       }
     }
@@ -1076,7 +1089,7 @@ int main(int argc, char **argv) {
       output_file = safe_fopen(output_filename, "w", "result mbox");
     } else {
       output_file = stdout;
-      quiet = 1;
+      global_quiet = 1;
     }
 
     if(nb_search_conditions > 0) {
@@ -1128,7 +1141,7 @@ int main(int argc, char **argv) {
       }
     }
 
-    if(!quiet) {
+    if(!global_quiet) {
       if(nb_extracted_mails > 0) {
         printf("Found %d matching mails.\n", nb_extracted_mails);
       } else {
@@ -1153,7 +1166,7 @@ int main(int argc, char **argv) {
   free(db_filename_list);
   free(mbox_filename_regexp_string);
 
-  regfree(&leading_from_line_regexp);
+  regfree(&global_leading_from_line_regexp);
 
   exit(EXIT_SUCCESS);
 }