13/4fbf9d886b164503c184041baa97e7f8337816

   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 641A6431FBD;\r
   6         Tue,  1 Dec 2009 19:01:45 -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 SULmnQ6B-1oe; Tue,  1 Dec 2009 19:01:43 -0800 (PST)\r
  11 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 10EFE431FAE;\r
  13         Tue,  1 Dec 2009 19:01:43 -0800 (PST)\r
  14 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  15         id 66B9625438C; Tue,  1 Dec 2009 19:01:25 -0800 (PST)\r
  16 From: Carl Worth <cworth@cworth.org>\r
  17 To: djcb@djcbsoftware.nl\r
  18 In-Reply-To: <87638qbkbo.wl%djcb@djcbsoftware.nl>\r
  19 References: <87r5rpyd4x.wl%djcb@djcbsoftware.nl>\r
  20         <87k4x6ad94.fsf@yoom.home.cworth.org>\r
  21         <87638qbkbo.wl%djcb@djcbsoftware.nl>\r
  22 Date: Tue, 01 Dec 2009 19:01:24 -0800\r
  23 Message-ID: <87aay29j0b.fsf@yoom.home.cworth.org>\r
  24 MIME-Version: 1.0\r
  25 Content-Type: multipart/signed; boundary="=-=-=";\r
  26         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  27 Cc: "notmuch@notmuchmail org" <notmuch@notmuchmail.org>\r
  28 Subject: Re: [notmuch] [PATCH 2/2] * avoid gcc 4.4.1 compiler warning due to\r
  29  ignored 'fflush' return value\r
  30 X-BeenThere: notmuch@notmuchmail.org\r
  31 X-Mailman-Version: 2.1.12\r
  32 Precedence: list\r
  33 List-Id: "Use and development of the notmuch mail system."\r
  34         <notmuch.notmuchmail.org>\r
  35 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  36         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  37 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  38 List-Post: <mailto:notmuch@notmuchmail.org>\r
  39 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  40 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  41         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  42 X-List-Received-Date: Wed, 02 Dec 2009 03:01:45 -0000\r
  43 \r
  44 --=-=-=\r
  45 \r
  46 On Tue, 01 Dec 2009 20:50:03 +0200, Dirk-Jan C. Binnema <djcb.bulk@gmail.com> wrote:\r
  47 > Sure, that's the better solution, but note that my patch did not introduce the\r
  48 > undefined behavior -- it was there before. I was trying a minimal patch to\r
  49 > silencing the warning.\r
  50 \r
  51 Yes, the leak was my bug.\r
  52 \r
  53 And the warning was useful to point to the bug. That's why silencing it\r
  54 would have been a bad thing.\r
  55 \r
  56 > Note that prompt seems to leak a bit, even after the\r
  57 > committed patch; attached are two more micro patches to fix this and another\r
  58 > small leak. I try to do minimal changes, but the prompt business gets a bit\r
  59 > unwieldy. The leaks are one-time at not critical, but anyway it's always good\r
  60 > stay vigilant.\r
  61 \r
  62 Yes, leak fixes are always appreciated.\r
  63 \r
  64 Could you resend these as complete git commits? I see only git diffs\r
  65 which lack both authorship and commit messages.\r
  66 \r
  67 Also, if you can send patches as text/plain rather than\r
  68 application/octet-stream then it's much easier to review them by simply\r
  69 replying and having the patch be quoted in the reply.\r
  70 \r
  71 Thanks,\r
  72 \r
  73 -Carl\r
  74 \r
  75 --=-=-=\r
  76 Content-Type: application/pgp-signature\r
  77 \r
  78 -----BEGIN PGP SIGNATURE-----\r
  79 Version: GnuPG v1.4.10 (GNU/Linux)\r
  80 \r
  81 iD8DBQFLFdiF6JDdNq8qSWgRAi8AAJ40iKiiXGyQ+Kr5A+eoF1ZSg4OH8QCeM3oI\r
  82 vd4FfixyUdtRMWVmlPoPxng=\r
  83 =3Q1G\r
  84 -----END PGP SIGNATURE-----\r
  85 --=-=-=--\r