43/275088d224f826747f7e236bcc0518cc155a53

   1 Return-Path: <david@tethera.net>\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 B1F0F431FB6\r
   6         for <notmuch@notmuchmail.org>; Sat,  5 Jul 2014 06:19:57 -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 pYTWlS60RAPq for <notmuch@notmuchmail.org>;\r
  16         Sat,  5 Jul 2014 06:19:53 -0700 (PDT)\r
  17 Received: from mx.xen14.node3324.gplhost.com (gitolite.debian.net\r
  18         [87.98.215.224]) (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id DCC20431FAF\r
  21         for <notmuch@notmuchmail.org>; Sat,  5 Jul 2014 06:19:52 -0700 (PDT)\r
  22 Received: from remotemail by mx.xen14.node3324.gplhost.com with local (Exim\r
  23         4.72) (envelope-from <david@tethera.net>)\r
  24         id 1X3PrE-0005vm-TV; Sat, 05 Jul 2014 13:18:24 +0000\r
  25 Received: (nullmailer pid 25817 invoked by uid 1000); Sat, 05 Jul 2014\r
  26         13:18:05 -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 v2 06/10] cli: refactor insert\r
  30 In-Reply-To: <1397653165-15620-7-git-send-email-novalazy@gmail.com>\r
  31 References: <1397653165-15620-1-git-send-email-novalazy@gmail.com>\r
  32         <1397653165-15620-7-git-send-email-novalazy@gmail.com>\r
  33 User-Agent: Notmuch/0.18.1+22~gbf82697 (http://notmuchmail.org) Emacs/24.3.1\r
  34         (x86_64-pc-linux-gnu)\r
  35 Date: Sat, 05 Jul 2014 10:18:05 -0300\r
  36 Message-ID: <87simgq702.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, 05 Jul 2014 13:19:57 -0000\r
  52 \r
  53 Peter Wang <novalazy@gmail.com> writes:\r
  54 \r
  55 > -    cleanup_path = tmppath;\r
  56 > -\r
  57 > -    if (! copy_stdin (fdin, fdout))\r
  58 > -     goto FAIL;\r
  59 > +    if (! copy_stdin (fdin, fdout)) {\r
  60 > +     close (fdout);\r
  61 > +     unlink (tmppath);\r
  62 > +     return FALSE;\r
  63 > +    }\r
  64 \r
  65 I'm not completely convinced by replacement of the "goto FAIL" with the\r
  66 multiple returns.  I'd lean to towards being consistent with the notmuch\r
  67 codebase unless the FAIL block is really horrendous\r
  68 \r
  69 Is there a good reason to use TRUE and FALSE for return values rather\r
  70 than EXIT_SUCCESS and EXIT_FAILURE? It seems like the latter would be\r
  71 overall slightly simpler in notmuch_insert_command.\r
  72 \r
  73 d\r