50/9795bcd15af963e6a3c718698ab154e6e1af77

   1 Return-Path: <amdragon@mit.edu>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 4FA6C429E36\r
   6         for <notmuch@notmuchmail.org>; Thu, 24 May 2012 15:01:24 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id ZSORz-6Iq3Uv for <notmuch@notmuchmail.org>;\r
  16         Thu, 24 May 2012 15:01:23 -0700 (PDT)\r
  17 Received: from dmz-mailsec-scanner-1.mit.edu (DMZ-MAILSEC-SCANNER-1.MIT.EDU\r
  18         [18.9.25.12])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 50A9C431FCF\r
  20         for <notmuch@notmuchmail.org>; Thu, 24 May 2012 15:01:23 -0700 (PDT)\r
  21 X-AuditID: 1209190c-b7fad6d000000920-9c-4fbeafb2b975\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 34.7B.02336.2BFAEBF4; Thu, 24 May 2012 18:01:23 -0400 (EDT)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id q4OM1Mf4018143; \r
  27         Thu, 24 May 2012 18:01:22 -0400\r
  28 Received: from drake.mit.edu (26-4-163.dynamic.csail.mit.edu [18.26.4.163])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q4OM1LnN009160\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Thu, 24 May 2012 18:01:22 -0400 (EDT)\r
  34 Received: from amthrax by drake.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1SXg5w-00064Z-2R; Thu, 24 May 2012 18:01:20 -0400\r
  37 From: Austin Clements <amdragon@MIT.EDU>\r
  38 To: notmuch@notmuchmail.org\r
  39 Subject: [PATCH v4 3/4] new: Merge error checks from add_files and\r
  40         add_files_recursive\r
  41 Date: Thu, 24 May 2012 18:01:12 -0400\r
  42 Message-Id: <1337896873-23253-4-git-send-email-amdragon@mit.edu>\r
  43 X-Mailer: git-send-email 1.7.10\r
  44 In-Reply-To: <1337896873-23253-1-git-send-email-amdragon@mit.edu>\r
  45 References: <1337887922-13163-1-git-send-email-amdragon@mit.edu>\r
  46         <1337896873-23253-1-git-send-email-amdragon@mit.edu>\r
  47 X-Brightmail-Tracker:\r
  48  H4sIAAAAAAAAA+NgFtrEIsWRmVeSWpSXmKPExsUixCmqrbt5/T5/gz3HmCyu35zJ7MDo8WzV\r
  49         LeYAxigum5TUnMyy1CJ9uwSujMNHdjEWHOavmNO0k72BcRJPFyMnh4SAicTWaxMYIWwxiQv3\r
  50         1rN1MXJxCAnsY5TY/mUOE4SzgVHi5ckzUM5xJolzF5qgymYxSjz+No0NpJ9NQENi2/7lYLNE\r
  51         BKQldt6dzdrFyMHBLKAm8adLBSQsLBAmsenLPGYQm0VAVeLErx1MICW8Ag4Sj/7yQlwhL/H0\r
  52         fh/YRE4BR4nHZ3+BTRQSKJe4fGMuywRG/gWMDKsYZVNyq3RzEzNzilOTdYuTE/PyUot0DfVy\r
  53         M0v0UlNKNzGCg0aSZwfjm4NKhxgFOBiVeHiZo/f5C7EmlhVX5h5ilORgUhLl/bEaKMSXlJ9S\r
  54         mZFYnBFfVJqTWnyIUYKDWUmEVzASKMebklhZlVqUD5OS5mBREuddqbnbX0ggPbEkNTs1tSC1\r
  55         CCYrw8GhJMF7cB1Qo2BRanpqRVpmTglCmomDE2Q4D9DwzyA1vMUFibnFmekQ+VOMilLivE9A\r
  56         EgIgiYzSPLheWFS/YhQHekWY9ztIFQ8wIcB1vwIazAQ0eMvjvSCDSxIRUlINjJ5lBlsTbZ/d\r
  57         yY7hmL+R5ygj049krsSLz1NNJa97KsZxPjbvufv44uuFArrNj6SU1xg0O25csKNPfNv2JUcv\r
  58         Cu7tOdtSu0Vsx6foC4lH+tZwWLPwX6rRZI29vnwL+0ZO6055ra4/MZev7n3o6C980mi59P+d\r
  59         Bz6l7D172y3nQMOO9A/XdZZpNSuxFGckGmoxFxUnAgBrK8sVxQIAAA==\r
  60 X-BeenThere: notmuch@notmuchmail.org\r
  61 X-Mailman-Version: 2.1.13\r
  62 Precedence: list\r
  63 List-Id: "Use and development of the notmuch mail system."\r
  64         <notmuch.notmuchmail.org>\r
  65 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  67 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  68 List-Post: <mailto:notmuch@notmuchmail.org>\r
  69 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  70 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  71         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  72 X-List-Received-Date: Thu, 24 May 2012 22:01:24 -0000\r
  73 \r
  74 Previously, add_files_recursive could have been called on a symlink to\r
  75 a non-directory.  Hence, calling it on a non-directory was not an\r
  76 error, so a separate function, add_files, existed to fail loudly in\r
  77 situations where the path had to be a directory.\r
  78 \r
  79 With the new stat-ing logic, add_files_recursive is always called on\r
  80 directories, so the separation of this logic is no longer necessary.\r
  81 Hence, this patch moves the strict error checking previously done by\r
  82 add_files into add_files_recursive.\r
  83 ---\r
  84  notmuch-new.c |   27 +++++----------------------\r
  85  1 file changed, 5 insertions(+), 22 deletions(-)\r
  86 \r
  87 diff --git a/notmuch-new.c b/notmuch-new.c\r
  88 index c64f1a7..2b05605 100644\r
  89 --- a/notmuch-new.c\r
  90 +++ b/notmuch-new.c\r
  91 @@ -308,11 +308,10 @@ add_files_recursive (notmuch_database_t *notmuch,\r
  92      }\r
  93      stat_time = time (NULL);\r
  94  \r
  95 -    /* This is not an error since we may have recursed based on a\r
  96 -     * symlink to a regular file, not a directory, and we don't know\r
  97 -     * that until this stat. */\r
  98 -    if (! S_ISDIR (st.st_mode))\r
  99 -       return NOTMUCH_STATUS_SUCCESS;\r
 100 +    if (! S_ISDIR (st.st_mode)) {\r
 101 +       fprintf (stderr, "Error: %s is not a directory.\n", path);\r
 102 +       return NOTMUCH_STATUS_FILE_ERROR;\r
 103 +    }\r
 104  \r
 105      fs_mtime = st.st_mtime;\r
 106  \r
 107 @@ -655,23 +654,7 @@ add_files (notmuch_database_t *notmuch,\r
 108            const char *path,\r
 109            add_files_state_t *state)\r
 110  {\r
 111 -    notmuch_status_t status;\r
 112 -    struct stat st;\r
 113 -\r
 114 -    if (stat (path, &st)) {\r
 115 -       fprintf (stderr, "Error reading directory %s: %s\n",\r
 116 -                path, strerror (errno));\r
 117 -       return NOTMUCH_STATUS_FILE_ERROR;\r
 118 -    }\r
 119 -\r
 120 -    if (! S_ISDIR (st.st_mode)) {\r
 121 -       fprintf (stderr, "Error: %s is not a directory.\n", path);\r
 122 -       return NOTMUCH_STATUS_FILE_ERROR;\r
 123 -    }\r
 124 -\r
 125 -    status = add_files_recursive (notmuch, path, state);\r
 126 -\r
 127 -    return status;\r
 128 +    return add_files_recursive (notmuch, path, state);\r
 129  }\r
 130  \r
 131  /* XXX: This should be merged with the add_files function since it\r
 132 -- \r
 133 1.7.10\r
 134 \r