ff/cc87493e007ddef94e1306402b1ccba7938868

   1 Return-Path: <bremner@unb.ca>\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 DAEC8431FC2\r
   6         for <notmuch@notmuchmail.org>; Fri, 29 Mar 2013 18:48:52 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 xL8nUo9RvUfg for <notmuch@notmuchmail.org>;\r
  16         Fri, 29 Mar 2013 18:48:52 -0700 (PDT)\r
  17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 915ED431FBF\r
  21         for <notmuch@notmuchmail.org>; Fri, 29 Mar 2013 18:48:52 -0700 (PDT)\r
  22 Received: from remotemail by tesseract.cs.unb.ca with local (Exim 4.80)\r
  23         (envelope-from <bremner@unb.ca>)\r
  24         id 1ULkuZ-000616-AF; Fri, 29 Mar 2013 22:48:51 -0300\r
  25 Received: (nullmailer pid 15258 invoked by uid 1000);\r
  26         Fri, 29 Mar 2013 23:59:56 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Peter Wang <novalazy@gmail.com>, notmuch@notmuchmail.org\r
  29 Subject: Re: [PATCH v4 06/12] test: add tests for insert\r
  30 In-Reply-To: <1359029288-12132-7-git-send-email-novalazy@gmail.com>\r
  31 References: <1359029288-12132-1-git-send-email-novalazy@gmail.com>\r
  32         <1359029288-12132-7-git-send-email-novalazy@gmail.com>\r
  33 User-Agent: Notmuch/0.15.1 (http://notmuchmail.org) Emacs/24.2.1\r
  34         (x86_64-pc-linux-gnu)\r
  35 Date: Fri, 29 Mar 2013 19:59:56 -0400\r
  36 Message-ID: <87d2uhiwoz.fsf@maritornes.cs.unb.ca>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Sat, 30 Mar 2013 01:48:53 -0000\r
  52 \r
  53 \r
  54 It took longer than I thought (of course) but I finally finished looking\r
  55 at the first 6 patches. \r
  56 \r
  57 I already mentioned a minor man page issue in a seperate message.\r
  58 \r
  59 I took a second pass through 03/12, and I think I would prefer thethe\r
  60 control flow of insert_message be closer to the standard style in\r
  61 notmuch of using a return value variable and a single cleanup block at\r
  62 the end.  Reasonable people can disagree about issues of style, but in\r
  63 the end consistency of the code base is also important.\r
  64 \r
  65 d\r