79/dbfcc258d0dcb7bd3deadbfd87f91441bde3d7

   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 arlo.cworth.org (Postfix) with ESMTP id 26E626DE00C9\r
   6  for <notmuch@notmuchmail.org>; Mon, 25 Apr 2016 12:40:18 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.016\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.016 tagged_above=-999 required=5\r
  12  tests=[AWL=-0.005, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01]\r
  13  autolearn=disabled\r
  14 Received: from arlo.cworth.org ([127.0.0.1])\r
  15  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16  with ESMTP id kWqQcrXqOwEo for <notmuch@notmuchmail.org>;\r
  17  Mon, 25 Apr 2016 12:40:10 -0700 (PDT)\r
  18 Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197])\r
  19  by arlo.cworth.org (Postfix) with ESMTPS id 5A6106DE00DF\r
  20  for <notmuch@notmuchmail.org>; Mon, 25 Apr 2016 12:40:09 -0700 (PDT)\r
  21 Received: from remotemail by fethera.tethera.net with local (Exim 4.84)\r
  22  (envelope-from <david@tethera.net>)\r
  23  id 1aumMi-0002nX-8I; Mon, 25 Apr 2016 15:40:16 -0400\r
  24 Received: (nullmailer pid 6093 invoked by uid 1000);\r
  25  Mon, 25 Apr 2016 19:40:07 -0000\r
  26 From: David Bremner <david@tethera.net>\r
  27 To: David Binderman <dcb314@hotmail.com>,\r
  28  "notmuch@notmuchmail.org" <notmuch@notmuchmail.org>\r
  29 Subject: RE: notmuch-0.21 bug report\r
  30 In-Reply-To: <DUB128-W40FF39685EC08E151E7E5B9C620@phx.gbl>\r
  31 References: <DUB128-W37A050CCA0515311F04F009C620@phx.gbl>\r
  32  <87wpnlbllb.fsf@zancas.localnet>\r
  33  <DUB128-W40FF39685EC08E151E7E5B9C620@phx.gbl>\r
  34 User-Agent: Notmuch/0.21+99~gd93d377 (http://notmuchmail.org) Emacs/24.5.1\r
  35  (x86_64-pc-linux-gnu)\r
  36 Date: Mon, 25 Apr 2016 16:40:07 -0300\r
  37 Message-ID: <87shy9bhpk.fsf@zancas.localnet>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: text/plain\r
  40 X-BeenThere: notmuch@notmuchmail.org\r
  41 X-Mailman-Version: 2.1.20\r
  42 Precedence: list\r
  43 List-Id: "Use and development of the notmuch mail system."\r
  44  <notmuch.notmuchmail.org>\r
  45 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  46  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  47 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  48 List-Post: <mailto:notmuch@notmuchmail.org>\r
  49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  50 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  51  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  52 X-List-Received-Date: Mon, 25 Apr 2016 19:40:18 -0000\r
  53 \r
  54 David Binderman <dcb314@hotmail.com> writes:\r
  55 \r
  56 > Hello there,\r
  57 >\r
  58 > ----------------------------------------\r
  59 >> Hi David;\r
  60 >>\r
  61 >> Thanks for the report. fclose doesn't really make sense here; \r
  62 >\r
  63 > Surprising. There is a call to fclose for the success return,\r
  64 > so I just assumed that one would also be required for the error return.\r
  65 >\r
  66 \r
  67 Ah OK, I thought the fclose referred to the notmuch database, but it\r
  68 refers to the input file. So yes, there is techically a resource leakage\r
  69 here, but I'm not sure how important it is, since these returns are part\r
  70 of program shutdown. Is there any harm in having FILEs open for read\r
  71 when a program shuts down? On the other hand, I guess it would only take\r
  72 5 or 6 added lines of code if someone wanted to add our usual "goto\r
  73 DONE" path\r
  74 \r
  75 d\r