Re: [PATCH] Rename already used counter var i

Subject: Re: [PATCH] Rename already used counter var i

Date: Tue, 27 Apr 2010 10:18:30 -0700

To: Sebastian Spaeth, Notmuch developer list

Cc:

From: Carl Worth


> On Tue, 27 Apr 2010 14:57:25 +0200, Sebastian Spaeth <Sebastian@SSpaeth.de> wrote:
> 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?

I investigated this. The key was that the user had to have no configured
"other" email addresses, (or at least fewer than the fixed-length array
(2) being iterated over in the outer loop) in order to trigger the
infinite-loop bug.

I just pushed new test cases to master that pass, and verified that the
same test cases cause the code in 0.3 to enter an infinite loop.

-Carl

-- 
carl.d.worth@intel.com
part-000.sig (application/pgp-signature)

Thread: