Large Mailbox Append Fix

[David Carter]

On Mon, 24 Sep 2007, Ken Murchison wrote:

> Attached is my rewrite against CVS.  I removed duplicate code that 
> resulted from having one for() loop with a special case if() inside (It 
> not uses a for() loop nested inside a do() loop).  The patch doesn't 
> look simpler because my version of diff appears to be braindead.

You need an update_uidlast() call if the last batch is empty, as Bron 
suggests. I'm also not convinced that:

     } while (!r);

ever exits. I think that it should be:

     } while (!r && msgno <= mailbox->exists);

Updated patch against current 2.3cvs attached.

-- 
David Carter                             Email: David.Carter at ucs.cam.ac.uk
University Computing Service,            Phone: (01223) 334502
New Museums Site, Pembroke Street,       Fax:   (01223) 334679
Cambridge UK. CB2 3QH.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cyrus-largemailboxappend-cvs.diff
Type: text/x-patch
Size: 12482 bytes
Desc: 
Url : http://lists.andrew.cmu.edu/pipermail/cyrus-devel/attachments/20070925/bff1e1f3/attachment.bin