[PATCH] Rename already used counter var i
Carl Worth
cworth at cworth.org
Tue Apr 27 08:12:46 PDT 2010
On Tue, 27 Apr 2010 14:57:25 +0200, Sebastian Spaeth <Sebastian at SSpaeth.de> wrote:
> From: Aneesh Kumar K.V <aneesh.kumar at linux.vnet.ibm.com>
> i is already used in a for loop at this point, so using i here again
> broke notmuch-reply (it would just hang). Use j instead of i here.
Thanks for the fix.
> Aneesh noticed this bug and provided a fix for it. IMO it warrants a 0.3.1
> release. notmuch-reply hanging indefinitely is not going to make grumpy
> editors happy.
Definitely.
For a point release like 0.3.1 I'm going to insist on a test case that
demonstrates each regression being fixed. Does someone know what it
takes to make a mail trigger this bug?
> Also despite a documentation blurb above the main snippet of
> code, I find it pretty much black magic what is being done here. But then
> I don't really know C....
I find it pretty much black magic too. I rewrote one piece of
black-magic code late last night when we noticed it was causing crashes
and valgrind was complaining. I'll dive in and give this chunk of code
some similar treatment.
-Carl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100427/dc70a40b/attachment.pgp>
More information about the notmuch
mailing list