Bastien <b...@gnu.org> writes:

> Hi Achim,
>
> Achim Gratz <strom...@nexgo.de> writes:
>
>> Bastien writes:
>>> (Sorry, I pushed a fix for the compiler warning that your patch also
>>> solves.)
>>
>> Not in master, though?
>
> Yes -- pushed this morning.
>
>>> Achim Gratz <strom...@nexgo.de> writes:
>>>
>>>> Since the test suite doesn't really test confirmation at all, it would
>>>> be nice if someone with a good variety of Babel calls could check if the
>>>> number of confirmations stay the same with and without the patch.  I
>>>> don't think I have test cases that cover all possible combinations.
>>>
>>> Please let's go and apply this, that's the best way to have more
>>> people testing it.  Thanks!
>>
>> Please wait for Eric to chime in.
>
> All right.  Copying Eric for the heads-up.
>

After walking through this patch, it looks like it provides exactly the
correct behavior.  Please go ahead and apply these changes.

Thanks Achim for the patch!

-- 
Eric Schulte
http://cs.unm.edu/~eschulte

Reply via email to