5a/76ea2e0f6ea8ef1647d25a48e02ba14346bd0e

   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 CFA79431FBD;\r
   6         Fri,  4 Dec 2009 11:18:37 -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 3rudHfjU+-YI; Fri,  4 Dec 2009 11:18:37 -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 00078431FAE;\r
  13         Fri,  4 Dec 2009 11:18:36 -0800 (PST)\r
  14 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  15         id A784C2542FB; Fri,  4 Dec 2009 11:18:36 -0800 (PST)\r
  16 From: Carl Worth <cworth@cworth.org>\r
  17 To: "Dirk-Jan C. Binnema" <djcb.bulk@gmail.com>, notmuch@notmuchmail.org\r
  18 In-Reply-To: <1259737885-15103-2-git-send-email-djcb.bulk@gmail.com>\r
  19 References: <1259737885-15103-1-git-send-email-djcb.bulk@gmail.com>\r
  20         <1259737885-15103-2-git-send-email-djcb.bulk@gmail.com>\r
  21 Date: Fri, 04 Dec 2009 11:18:36 -0800\r
  22 Message-ID: <873a3qfsz7.fsf@yoom.home.cworth.org>\r
  23 MIME-Version: 1.0\r
  24 Content-Type: multipart/signed; boundary="=-=-=";\r
  25         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  26 Subject: Re: [notmuch] [PATCH 2/2] * free the response data from 'prompt'\r
  27 X-BeenThere: notmuch@notmuchmail.org\r
  28 X-Mailman-Version: 2.1.12\r
  29 Precedence: list\r
  30 List-Id: "Use and development of the notmuch mail system."\r
  31         <notmuch.notmuchmail.org>\r
  32 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  33         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  34 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  35 List-Post: <mailto:notmuch@notmuchmail.org>\r
  36 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  37 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  38         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  39 X-List-Received-Date: Fri, 04 Dec 2009 19:18:38 -0000\r
  40 \r
  41 --=-=-=\r
  42 Content-Transfer-Encoding: quoted-printable\r
  43 \r
  44 On Wed,  2 Dec 2009 09:11:25 +0200, "Dirk-Jan C. Binnema" <djcb.bulk@gmail.=\r
  45 com> wrote:\r
  46 > Free the results of the prompt; this patch does the minimal job for that.\r
  47 > It may be nice to refactor the function a bit.=20\r
  48 >=20\r
  49 > Signed-off-by: Dirk-Jan C. Binnema <djcb.bulk@gmail.com>\r
  50 \r
  51 Hi there,\r
  52 \r
  53 I pushed the first leak fix from this series, but the below is doing a\r
  54 little more work than necessary.\r
  55 \r
  56 The getline function is happy to accept a malloc'ed pointer and return\r
  57 it again if it's large enough, (or otherwise realloc it and return the\r
  58 result).\r
  59 \r
  60 So we don't need to free response between each call to prompt, but just\r
  61 after the last one.\r
  62 \r
  63 =2DCarl\r
  64 \r
  65 --=-=-=\r
  66 Content-Type: application/pgp-signature\r
  67 \r
  68 -----BEGIN PGP SIGNATURE-----\r
  69 Version: GnuPG v1.4.10 (GNU/Linux)\r
  70 \r
  71 iD8DBQFLGWCM6JDdNq8qSWgRAoaZAJ9YZbpPSD3cKgaDY01R1etohqC7jQCaAsEx\r
  72 WaxR8cRlndvYdHewqPiLs1k=\r
  73 =zLTo\r
  74 -----END PGP SIGNATURE-----\r
  75 --=-=-=--\r