c4/6014f854e9dd715e38dd61d3bd50b905e5f5af

   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 900B54196F2\r
   6         for <notmuch@notmuchmail.org>; Tue, 27 Apr 2010 08:12:59 -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: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id k8V01wTbfqhL; Tue, 27 Apr 2010 08:12:58 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id C4781431FC1;\r
  19         Tue, 27 Apr 2010 08:12:58 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id 6602125400A; Tue, 27 Apr 2010 08:12:58 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: Sebastian Spaeth <Sebastian@SSpaeth.de>,\r
  24         Notmuch developer list <notmuch@notmuchmail.org>\r
  25 Subject: Re: [PATCH] Rename already used counter var i\r
  26 In-Reply-To: <1272373045-29268-1-git-send-email-Sebastian@SSpaeth.de>\r
  27 References: <87tyqxmdw6.fsf@yoom.home.cworth.org>\r
  28         <1272373045-29268-1-git-send-email-Sebastian@SSpaeth.de>\r
  29 User-Agent: Notmuch/0.2-251-g8514e59 (http://notmuchmail.org) Emacs/23.1.1\r
  30         (i486-pc-linux-gnu)\r
  31 Date: Tue, 27 Apr 2010 08:12:46 -0700\r
  32 Message-ID: <87r5m1lyj5.fsf@yoom.home.cworth.org>\r
  33 MIME-Version: 1.0\r
  34 Content-Type: multipart/signed; boundary="=-=-=";\r
  35         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  36 X-BeenThere: notmuch@notmuchmail.org\r
  37 X-Mailman-Version: 2.1.13\r
  38 Precedence: list\r
  39 List-Id: "Use and development of the notmuch mail system."\r
  40         <notmuch.notmuchmail.org>\r
  41 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  42         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  43 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  44 List-Post: <mailto:notmuch@notmuchmail.org>\r
  45 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  46 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  48 X-List-Received-Date: Tue, 27 Apr 2010 15:12:59 -0000\r
  49 \r
  50 --=-=-=\r
  51 Content-Transfer-Encoding: quoted-printable\r
  52 \r
  53 On Tue, 27 Apr 2010 14:57:25 +0200, Sebastian Spaeth <Sebastian@SSpaeth.de>=\r
  54  wrote:\r
  55 > From: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>\r
  56 > i is already used in a for loop at this point, so using i here again\r
  57 > broke notmuch-reply (it would just hang). Use j instead of i here.\r
  58 \r
  59 Thanks for the fix.\r
  60 \r
  61 >  Aneesh noticed this bug and provided a fix for it. IMO it warrants a 0.3=\r
  62 .1\r
  63 >  release. notmuch-reply hanging indefinitely is not going to make grumpy\r
  64 >  editors happy.\r
  65 \r
  66 \r
  67 Definitely.\r
  68 \r
  69 For a point release like 0.3.1 I'm going to insist on a test case that\r
  70 demonstrates each regression being fixed. Does someone know what it\r
  71 takes to make a mail trigger this bug?\r
  72 \r
  73 >  Also despite a documentation blurb above the main snippet of\r
  74 >  code, I find it pretty much black magic what is being done here. But the=\r
  75 n=20\r
  76 >  I don't really know C....\r
  77 \r
  78 I find it pretty much black magic too. I rewrote one piece of\r
  79 black-magic code late last night when we noticed it was causing crashes\r
  80 and valgrind was complaining. I'll dive in and give this chunk of code\r
  81 some similar treatment.\r
  82 \r
  83 =2DCarl\r
  84 \r
  85 --=-=-=\r
  86 Content-Type: application/pgp-signature\r
  87 \r
  88 -----BEGIN PGP SIGNATURE-----\r
  89 Version: GnuPG v1.4.10 (GNU/Linux)\r
  90 \r
  91 iD8DBQFL1v7u6JDdNq8qSWgRAgQGAJ9xJh55o3PALeJrCiUcsogvrZ6MiQCfcFbg\r
  92 S99Z/jI645zQTcMmuM60yiE=\r
  93 =LhWS\r
  94 -----END PGP SIGNATURE-----\r
  95 --=-=-=--\r