22/cfe407f7ffd78088db79c7335f744aee3a6cbb

   1 Return-Path: <cworth@cworth.org>\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 C097B431FC0;\r
   6         Fri, 27 Nov 2009 19:41:51 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id 5jfg2do4YyYS; Fri, 27 Nov 2009 19:41:51 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id B9E0F431FAE;\r
  13         Fri, 27 Nov 2009 19:41:50 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: Jan Janak <jan@ryngle.com>, notmuch@notmuchmail.org\r
  16 In-Reply-To: <1259183505-30840-1-git-send-email-jan@ryngle.com>\r
  17 References: <1259178195-30324-1-git-send-email-jan@ryngle.com>\r
  18         <1259183505-30840-1-git-send-email-jan@ryngle.com>\r
  19 Date: Fri, 27 Nov 2009 19:41:36 -0800\r
  20 Message-ID: <87vdgvjoy7.fsf@yoom.home.cworth.org>\r
  21 MIME-Version: 1.0\r
  22 Content-Type: text/plain; charset=us-ascii\r
  23 Subject: Re: [notmuch] [PATCH] notmuch-new: Test if directory looks like\r
  24  Maildir before skipping tmp.\r
  25 X-BeenThere: notmuch@notmuchmail.org\r
  26 X-Mailman-Version: 2.1.12\r
  27 Precedence: list\r
  28 List-Id: "Use and development of the notmuch mail system."\r
  29         <notmuch.notmuchmail.org>\r
  30 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  31         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  32 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  33 List-Post: <mailto:notmuch@notmuchmail.org>\r
  34 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  35 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  36         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  37 X-List-Received-Date: Sat, 28 Nov 2009 03:41:51 -0000\r
  38 \r
  39 On Wed, 25 Nov 2009 22:11:45 +0100, Jan Janak <jan@ryngle.com> wrote:\r
  40 > Files and subdirectories with the name "tmp" that do not look like\r
  41 > Maildir will still be processed by 'notmuch new'.\r
  42 \r
  43 This is a nice improvement. I realize we neglected to document the\r
  44 original "tmp" skipping. This should definitely be safer, (though we\r
  45 should document this as well).\r
  46 \r
  47 And this maildir detection will also be useful for the outstanding patch\r
  48 to add tags based on flags in maildir filenames.\r
  49 \r
  50 I pushed the patch and then made two, minor whitespace fixes:\r
  51 \r
  52 > +static int is_maildir (struct dirent **entries, int count)\r
  53 \r
  54 The function name in the definition here belongs on its own line.\r
  55 \r
  56 > +         if (found == 3) return 1;\r
  57 \r
  58 Having the action of the "if" statement on the same line as the "if"\r
  59 itself hides it too much. That also belongs on the next line.\r
  60 \r
  61 -Carl\r